Reviewers: Benedikt Meurer,
Description:
[turbofan] Float32 LinkageLocations need double registers too.
R=bmeu...@chromium.org
BUG=
Please review this at https://codereview.chromium.org/1268433003/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+2, -1 lines):
M src/compiler/instruction-selector-impl.h
Index: src/compiler/instruction-selector-impl.h
diff --git a/src/compiler/instruction-selector-impl.h
b/src/compiler/instruction-selector-impl.h
index
db9e100e018b429a8718475d348a795b51292035..50a94342c9abb97e16f853f79747615a3fb5d951
100644
--- a/src/compiler/instruction-selector-impl.h
+++ b/src/compiler/instruction-selector-impl.h
@@ -224,7 +224,8 @@ class OperandGenerator {
location.AsCalleeFrameSlot(),
virtual_register);
}
// a fixed register.
- if (RepresentationOf(type) == kRepFloat64) {
+ MachineType rep = RepresentationOf(type);
+ if (rep == kRepFloat64 || rep == kRepFloat32) {
return UnallocatedOperand(UnallocatedOperand::FIXED_DOUBLE_REGISTER,
location.AsRegister(), virtual_register);
}
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.