Request for review (L): 6797305: Add LoadUB and LoadUI opcode class

Christian Thalinger Christian.Thalinger at Sun.COM
Fri Mar 6 11:31:29 PST 2009


On Fri, 2009-03-06 at 10:18 -0800, Vladimir Kozlov wrote:
> Matcher definitely should not do this but it could be confused since
> all LoadB mem, LoadUB mem, LoadI mem produce the same RegI register
> and it takes first one defined in generated/adfiles/dfa_x86_64.cpp
> in State::_sub_Op_ConvI2L(const Node *n) method.

Maybe the problem this (in ad_x86_64.hpp):

class loadB2LNode : public MachNode { 
...
  virtual int            ideal_Opcode() const { return Op_ConvI2L; }

But I'm just guessing...

-- Christian




More information about the hotspot-compiler-dev mailing list