Updates:
Cc: [email protected] [email protected]
Comment #2 on issue 2124 by [email protected]: Register Allocator in
some condition doesn't handle FixedTemp correctly
http://code.google.com/p/v8/issues/detail?id=2124
Thanks for the report! We'll have a look. Unwanted aliasing can indeed be a
problem.
After a quick glance at current V8 this problem may not (yet) occur because
we don't have an instruction that requests a input-temp (UseTempRegister)
and a fixed-temp (FixedTemp) at the same time, but I agree that this
scenario should be supported correctly by the allocator.
Sidenote: Ideally, I think LOperands should be passed by value only and we
should not have LOperand* in LIR instructions, but embedded LOperand
instances instead. I tried refactoring our LIR once, but it turns out to be
a massive change to all lithium* files. I still think this would be a good
idea!
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
