RFR(M): 8200303: C2 should leverage profiling for lookupswitch/tableswitch
Roland Westrelin
rwestrel at redhat.com
Wed Apr 4 15:44:56 UTC 2018
Hi Tobias,
Thanks for reviewing this.
> machnode.hpp
> - Do we need a forward declaration of MachJumpNode at the beginning of the header file (like we do
> have for the other MachNodes)?
Not sure why those are there but I don't see a need for one for
MachJumpNode.
> - I think you also need to define MachJumpNode in vmStructs.cpp
Right. I noticed the SA has a MachIfNode class so I suppose it need a
MachJumNode class too. I need to take a closer look.
> parse2.cpp
> - I would prefer "always/never taken" over "taken always/never"
I re-used the text from Parse::adjust_map_after_if().
> - 1160: what is 'ifff' used for?
The ifff is not something new. I changed the arguments that
jump_if_fork_int() takes so had to adjust that line of code.
Roland.
More information about the hotspot-compiler-dev
mailing list