RFR (S): 8150247: CompilerControl: LogCompilation testing

Pavel Punegov pavel.punegov at oracle.com
Fri May 6 19:35:55 UTC 2016


I have updated webrev to add @module to jtreg header
http://cr.openjdk.java.net/~ppunegov/8150247/webrev.01/ <http://cr.openjdk.java.net/~ppunegov/8150247/webrev.01/>

— Pavel.

> On 05 May 2016, at 21:06, Pavel Punegov <pavel.punegov at oracle.com> wrote:
> Hi,
> please review the following change to CompilerControl tests. It adds a new test to cover the gap found by JDK-8145345 [*].
> The LogCompilation was empty after CompilerControl was integrated, but tests haven’t found that. This happened only when the LogCompilation was specified while there were no directives/CompileCommands enabled. With enabled directive/command it worked fine.
> Tests didn’t found the issue due to:
> 1. There were no standalone test for LogCompilation. All related tests enable appropriate log directive or command.
> 2. LogProcessor incorrectly worked for empty tasks that should be logged. It didn’t reported the issue.
> [*] https://bugs.openjdk.java.net/browse/JDK-8145345 <https://bugs.openjdk.java.net/browse/JDK-8145345>
> bug: https://bugs.openjdk.java.net/browse/JDK-8150247 <https://bugs.openjdk.java.net/browse/JDK-8150247>
> webrev: http://cr.openjdk.java.net/~ppunegov/8150247/webrev.00/ <http://cr.openjdk.java.net/~ppunegov/8150247/webrev.00/>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20160506/d26385ee/attachment-0001.html>

More information about the hotspot-compiler-dev mailing list