Request for reviews (S): 7200233: C2: can't use expand rules for vector instruction rules (original) (raw)

Christian Thalinger christian.thalinger at oracle.com
Mon Sep 24 17:27:42 PDT 2012


On Sep 21, 2012, at 4:43 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:

http://cr.openjdk.java.net/~kvn/7200233/webrev

Added missed bottomtype set in ArchDesc::defineExpand(). Added missed vector nodes in MatchRule::isvector() and in vmStructs.

Looks good. But wouldn't it be better to have some kind of array with all the vector names in it for MatchRule::is_vector?

-- Chris

Thanks, Vladimir



More information about the hotspot-compiler-dev mailing list