Commit 60c0389b authored by jyan's avatar jyan Committed by Commit bot

S390: Fix Macro-Asm DCHECK

Move DCHECK to only check for necessary path.

R=joransiu@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com
BUG=

Review-Url: https://codereview.chromium.org/1946003003
Cr-Commit-Position: refs/heads/master@{#36034}
parent 04c8c11e
......@@ -5215,12 +5215,12 @@ void MacroAssembler::ShiftRight(Register dst, Register src,
// Shift right logical for 32-bit integer types.
void MacroAssembler::ShiftRight(Register dst, Register src, Register val) {
DCHECK(!dst.is(val)); // The lr/srl path clobbers val.
if (dst.is(src)) {
srl(dst, val);
} else if (CpuFeatures::IsSupported(DISTINCT_OPS)) {
srlk(dst, src, val);
} else {
DCHECK(!dst.is(val)); // The lr/srl path clobbers val.
lr(dst, src);
srl(dst, val);
}
......@@ -5241,12 +5241,12 @@ void MacroAssembler::ShiftLeftArith(Register dst, Register src,
// Shift left arithmetic for 32-bit integer types.
void MacroAssembler::ShiftLeftArith(Register dst, Register src, Register val) {
DCHECK(!dst.is(val)); // The lr/sla path clobbers val.
if (dst.is(src)) {
sla(dst, val);
} else if (CpuFeatures::IsSupported(DISTINCT_OPS)) {
slak(dst, src, val);
} else {
DCHECK(!dst.is(val)); // The lr/sla path clobbers val.
lr(dst, src);
sla(dst, val);
}
......@@ -5267,12 +5267,12 @@ void MacroAssembler::ShiftRightArith(Register dst, Register src,
// Shift right arithmetic for 32-bit integer types.
void MacroAssembler::ShiftRightArith(Register dst, Register src, Register val) {
DCHECK(!dst.is(val)); // The lr/sra path clobbers val.
if (dst.is(src)) {
sra(dst, val);
} else if (CpuFeatures::IsSupported(DISTINCT_OPS)) {
srak(dst, src, val);
} else {
DCHECK(!dst.is(val)); // The lr/sra path clobbers val.
lr(dst, src);
sra(dst, val);
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment