RFR(S): remove ExecuteInternalVMTests and VerboseInternalVMTests flags (original) (raw)
Igor Ignatyev igor.ignatyev at oracle.com
Thu Nov 1 23:23:03 UTC 2018
- Previous message (by thread): RFR(S): remove ExecuteInternalVMTests and VerboseInternalVMTests flags
- Next message (by thread): RFR(S): remove ExecuteInternalVMTests and VerboseInternalVMTests flags
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Hi David,
removing usage of test_log will just mix "unneeded" changes w/ this clean up. as TestReservedSpace_test, TestReserveMemorySpecial_test, TestVirtualSpace_test, and TestMetaspaceUtils_test are to be removed by 8213269[*], I don't think we need to pay much attention to their code.
[*] https://bugs.openjdk.java.net/browse/JDK-8213269 <https://bugs.openjdk.java.net/browse/JDK-8213269>
-- Igor
On Nov 1, 2018, at 4:16 PM, David Holmes <david.holmes at oracle.com> wrote:
Hi Igor, There's no point having empty testlog macros that do nothing. The macro and all uses should just be deleted ... unless you plan on adding some other form of logging for this? Thanks, David On 2/11/2018 7:15 AM, Igor Ignatyev wrote: http://cr.openjdk.java.net/~iignatyev//8213058/webrev.00/index.html
174 lines changed: 0 ins; 170 del; 4 mod; Hi all, could you please review this small clean up which removes ExecuteInternalVMTests and VerboseInternalVMTests flags and related make targets and tests? 8177708[1-3] is to convert the last of internal vm tests, so the whole InternalVMTests can be removed. JBS: https://bugs.openjdk.java.net/browse/JDK-8213058 webrev: http://cr.openjdk.java.net/~iignatyev//8213058/webrev.00/index.html testing: tier1, build all regular platforms [1] https://bugs.openjdk.java.net/browse/JDK-8177708 [2] http://cr.openjdk.java.net/~iignatyev//8177708/webrev.00/index.html [3] http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2018-October/030633.html Thanks, -- Igor
- Previous message (by thread): RFR(S): remove ExecuteInternalVMTests and VerboseInternalVMTests flags
- Next message (by thread): RFR(S): remove ExecuteInternalVMTests and VerboseInternalVMTests flags
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]