RFR(M): 8135068: Extract methodmatchers from CompilerOracle

Nils Eliasson nils.eliasson at oracle.com
Mon Sep 7 12:58:06 UTC 2015


Hi all,

This is another preparatory change for compiler control. This change 
extracts the code for matching methods in CompilerOracle into a separate 
class that has subtypes for basic and typed matchers. (In a future 
change an additional subtype will show up.) The code is mostly the same 
but include fixes for error handling and pattern validation.

A whitebox method and a test (MethodMatcherTest.java) is supplied for 
ensuring correctness of the rewrite.

This change fixes the following bugs:

https://bugs.openjdk.java.net/browse/JDK-8074095 - Method validation in 
CompileCommand
https://bugs.openjdk.java.net/browse/JDK-8081387 - C2 does not respect 
exclu de command for inlining
https://bugs.openjdk.java.net/browse/JDK-8069173 - CompileCommand needs 
better test coverage

This patch may have to applied on top of 
http://cr.openjdk.java.net/~neliasso/8135067 Preparatory refactorings 
for compiler control to apply cleanly.

bug: https://bugs.openjdk.java.net/browse/JDK-8135068
webrev hotspot: http://c.openjdk.java.net/~neliasso/8135068/webrev.04
webrev jdk: http://c.openjdk.java.net/~neliasso/8135068/webrev_jdk.01

The change passes regular hotspot testing.

Thanks,
Nils
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20150907/9f4ca26e/attachment.html>


More information about the hotspot-compiler-dev mailing list