Re: RFR: JDK-8282797: CompileCommand parsing errors should exit VM [v3]

2023-05-31 Thread Vladimir Kozlov
On Wed, 24 May 2023 11:05:01 GMT, Tobias Holenstein wrote: >> Currently, errors during compile command parsing just print an error but >> don't exit the VM. As a result, issues go unnoticed. >> >> With this PR the behavior is changed to exit the VM when an error occurs. >> >> E.g. `java -XX:

Re: RFR: JDK-8282797: CompileCommand parsing errors should exit VM [v3]

2023-05-31 Thread Christian Hagedorn
On Wed, 24 May 2023 11:05:01 GMT, Tobias Holenstein wrote: >> Currently, errors during compile command parsing just print an error but >> don't exit the VM. As a result, issues go unnoticed. >> >> With this PR the behavior is changed to exit the VM when an error occurs. >> >> E.g. `java -XX:

Re: RFR: JDK-8282797: CompileCommand parsing errors should exit VM [v3]

2023-05-24 Thread Tobias Holenstein
On Wed, 24 May 2023 09:30:58 GMT, Tobias Hartmann wrote: >> Tobias Holenstein has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - Update Scenario.java >> - Update compilerOracle.cpp > > test/hotspot/jtreg/compiler/compilercontrol/share/sc

Re: RFR: JDK-8282797: CompileCommand parsing errors should exit VM [v3]

2023-05-24 Thread Tobias Holenstein
> Currently, errors during compile command parsing just print an error but > don't exit the VM. As a result, issues go unnoticed. > > With this PR the behavior is changed to exit the VM when an error occurs. > > E.g. `java -XX:CompileCommand=compileonly,HashMap:: -version` will exit the > VM a

Re: RFR: JDK-8282797: CompileCommand parsing errors should exit VM [v3]

2023-05-24 Thread Tobias Holenstein
On Wed, 24 May 2023 08:20:32 GMT, Christian Hagedorn wrote: >> Tobias Holenstein has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - Update Scenario.java >> - Update compilerOracle.cpp > > src/hotspot/share/compiler/compilerOracle.cpp li

Re: RFR: JDK-8282797: CompileCommand parsing errors should exit VM [v2]

2023-05-24 Thread Tobias Holenstein
On Wed, 24 May 2023 09:29:53 GMT, Tobias Hartmann wrote: >> Tobias Holenstein has updated the pull request incrementally with four >> additional commits since the last revision: >> >> - Update >> test/hotspot/jtreg/compiler/compilercontrol/share/scenario/Scenario.java >> >>Co-authored

Re: RFR: JDK-8282797: CompileCommand parsing errors should exit VM [v2]

2023-05-24 Thread Tobias Holenstein
> Currently, errors during compile command parsing just print an error but > don't exit the VM. As a result, issues go unnoticed. > > With this PR the behavior is changed to exit the VM when an error occurs. > > E.g. `java -XX:CompileCommand=compileonly,HashMap:: -version` will exit the > VM a

Re: RFR: JDK-8282797: CompileCommand parsing errors should exit VM [v2]

2023-05-24 Thread Tobias Holenstein
On Wed, 24 May 2023 09:29:10 GMT, Christian Hagedorn wrote: >> Tobias Holenstein has updated the pull request incrementally with four >> additional commits since the last revision: >> >> - Update >> test/hotspot/jtreg/compiler/compilercontrol/share/scenario/Scenario.java >> >>Co-auth

Re: RFR: JDK-8282797: CompileCommand parsing errors should exit VM

2023-05-24 Thread Tobias Hartmann
On Tue, 2 May 2023 11:35:54 GMT, Tobias Holenstein wrote: > Currently, errors during compile command parsing just print an error but > don't exit the VM. As a result, issues go unnoticed. > > With this PR the behavior is changed to exit the VM when an error occurs. > > E.g. `java -XX:Compile

Re: RFR: JDK-8282797: CompileCommand parsing errors should exit VM

2023-05-24 Thread Christian Hagedorn
On Tue, 2 May 2023 11:35:54 GMT, Tobias Holenstein wrote: > Currently, errors during compile command parsing just print an error but > don't exit the VM. As a result, issues go unnoticed. > > With this PR the behavior is changed to exit the VM when an error occurs. > > E.g. `java -XX:Compile

Re: RFR: JDK-8282797: CompileCommand parsing errors should exit VM

2023-05-23 Thread Vladimir Kozlov
On Tue, 2 May 2023 11:35:54 GMT, Tobias Holenstein wrote: > Currently, errors during compile command parsing just print an error but > don't exit the VM. As a result, issues go unnoticed. > > With this PR the behavior is changed to exit the VM when an error occurs. > > E.g. `java -XX:Compile