Commit 1827b842 authored by Clemens Hammacher's avatar Clemens Hammacher Committed by Commit Bot

[assembler][ia32] Fix redundant condition

Immediate::is_zero already checks the reloc info to be none, so the
additional check is redundant.

R=tebbi@chromium.org

Change-Id: I3ec91fe60e8c659b2f38fda0123784a69e4bcbe9
Reviewed-on: https://chromium-review.googlesource.com/878321Reviewed-by: 's avatarTobias Tebbi <tebbi@chromium.org>
Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#50760}
parent 25ecc45f
...@@ -1159,8 +1159,7 @@ void TurboAssembler::Move(Register dst, Register src) { ...@@ -1159,8 +1159,7 @@ void TurboAssembler::Move(Register dst, Register src) {
} }
void TurboAssembler::Move(Register dst, const Immediate& x) { void TurboAssembler::Move(Register dst, const Immediate& x) {
if (!x.is_heap_object_request() && x.is_zero() && if (!x.is_heap_object_request() && x.is_zero()) {
RelocInfo::IsNone(x.rmode())) {
xor_(dst, dst); // Shorter than mov of 32-bit immediate 0. xor_(dst, dst); // Shorter than mov of 32-bit immediate 0.
} else { } else {
mov(dst, x); mov(dst, x);
......
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