RFR(T) : 8251997 : remove usage of PropertyResolvingWrapper in vmTestbase/vm/mlvm/

Igor Ignatyev igor.ignatyev at oracle.com
Thu Sep 3 16:23:01 UTC 2020


Hi Tobias,

thanks for your prompt review, pushed.

-- Igor

> On Sep 3, 2020, at 5:08 AM, Tobias Hartmann <tobias.hartmann at oracle.com> wrote:
> 
> Hi Igor,
> 
> looks good and trivial to me.
> 
> Best regards,
> Tobias
> 
> On 03.09.20 00:18, Igor Ignatyev wrote:
>> http://cr.openjdk.java.net/~iignatyev//8251997/webrev.00/
>>> 27 lines changed: 5 ins; 12 del; 10 mod;
>> 
>> 
>> Hi all,
>> 
>> could you please review this small and trivial patch which removes usage of PropertyResolvingWrapper from vm/mlvm tests and reenables "smart action arguments"?
>> a bit of background from main bug (8219140):
>>> CODETOOLS-7902352 added support of using ${property} in action directive, so PropertyResolvingWrapper isn't needed anymore and can be removed. 
>> 
>> testing: vmTestbase/vm/mlvm/
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8251997
>> webrev: http://cr.openjdk.java.net/~iignatyev//8251997/webrev.00/
>> 
>> Thanks,
>> -- Igor
>> 



More information about the hotspot-compiler-dev mailing list