Good idea.
http://codereview.chromium.org/8086021/diff/1/src/ia32/assembler-ia32.h File src/ia32/assembler-ia32.h (right): http://codereview.chromium.org/8086021/diff/1/src/ia32/assembler-ia32.h#newcode350 src/ia32/assembler-ia32.h:350: bool is_reg_only() const; On 2011/09/30 13:42:20, Yang wrote:
Is this still necessary if Operand wrapper has been removed?
For now it is. Under the hood it's the same implementation as before, but I made the Operand(Register) constructor private. I think it would be a good idea to get rid of that constructor and clean up the encodings, but that's work for a separate change. http://codereview.chromium.org/8086021/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
