tree 3285c18d7b29bb818bedabae6120baae8b444321
parent 91ddef31f229e6d116cc8988b71aa77bc5083bcb
author Vladimir Marko <vmarko@google.com> 1540922995 +0000
committer Vladimir Marko <vmarko@google.com> 1540984384 +0000

Fix and improve ClinitCheck elimination.

Fix erroneous usage of the inner method's access flags
while checking the class of the outer method. This allowed
erroneous elimination of a needed ClinitCheck when inlining.

Treat constructors the same way as static methods as the
instance allocation necessarily preceding the constructor
call performs the class initialization check.

The size of the aosp_taimen-userdebug prebuilts:
 - before:
   arm/boot*.oat: 20252196
   arm64/boot*.oat: 24030776
   oat/arm64/services.odex: 22406664
 - after:
   arm/boot*.oat: 20252092 (-0.1KiB)
   arm64/boot*.oat: 24027024 (-3.7KiB)
   oat/arm64/services.odex: 22402528 (-4.0KiB)
(Insignificant changes.)

Test: Add regression test to 174-escaping-instance-of-bad-class.
Test: Add optimization test to 551-checker-clinit.
Test: testrunner.py --jvm -t 174
Test: m test-art-host-gtest
Test: testrunner.py --host --optimizing
Test: Pixel 2 XL boots.
Test: m test-art-target-gtest
Test: testrunner.py --target --optimizing
Bug: 62478025
Change-Id: I591aca2c538d10cf6df1d38d59270af1de380b3e