RFR: 8266088: compiler/arguments/TestPrintOptoAssemblyLineNumbers test should user driver mode [v3]

Igor Ignatyev iignatyev at openjdk.java.net
Tue Apr 27 20:30:35 UTC 2021


> Hi all,
> 
> could you please review this tiny patch for `TestPrintOptoAssemblyLineNumbers` test? 
> from JBS:
>> compiler/arguments/TestPrintOptoAssemblyLineNumbers class is a driver class that spawns a new JVM and validates its output, there is no need for the JVM which executes TestPrintOptoAssemblyLineNumbers to be run w/ all vm flags.
> 
> besides addressing this, the patch also removes `/compiler/patches` from `@library` as it's not needed and fixes code style here and there.
> 
> testing:
> - [x] `compiler/arguments/TestPrintOptoAssemblyLineNumbers` on `{linux,windows,macosx}-x64-debug`
> 
> Thanks,
> -- Igor

Igor Ignatyev has updated the pull request incrementally with one additional commit since the last revision:

  fixed line numbers

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/3726/files
  - new: https://git.openjdk.java.net/jdk/pull/3726/files/d4b342a9..b5a52b94

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3726&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3726&range=01-02

  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.java.net/jdk/pull/3726.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/3726/head:pull/3726

PR: https://git.openjdk.java.net/jdk/pull/3726


More information about the hotspot-compiler-dev mailing list