RFR (S) 8150534: C1 compilation fails with "Constant field loads are folded during parsing"

[Aleksey Shipilev]

Hi,

In JDK-8150102, we assumed that all suitable constants are exposed as
Constants after parsing, and added the assert to test that invariant is
true. And of course, it asserts in some corner cases which we cannot
reliably reproduce.

With no clear way to fix the underlying issue, we would like to
re-introduce the deleted branch in Canonicalizer:
  https://bugs.openjdk.java.net/browse/JDK-8150534
  http://cr.openjdk.java.net/~shade/8150534/webrev.01/

Note that it fixes the case when -XX:+PatchALot is used, which
definitely asserts. Some tests unreliably assert even with
out-of-the-box config. I forked the investigation about the Constant
field loads to:
  https://bugs.openjdk.java.net/browse/JDK-8150547

Testing: regression tests; JPRT (in progress)

Thanks,
-Aleksey

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20160224/b62b2539/signature.asc>