Re: [10] RFR: 8191813: compiler/runtime/SpreadNullArg.java fails in tier1

2017-11-26 Thread Rahul Raghavan
Thank you Mandy for review. On Friday 24 November 2017 11:11 PM, mandy chung wrote: Looks good.  Thanks for fixing the test, Rahul. Mandy On 11/24/17 3:16 AM, Rahul Raghavan wrote: Hi, Please review the following fix proposal for 8191813. webrev - http://cr.openjdk.java.net/~rraghavan/81918

Re: [10] RFR: 8191813: compiler/runtime/SpreadNullArg.java fails in tier1

2017-11-26 Thread Rahul Raghavan
Thank you Dan for the review. Rahul On Friday 24 November 2017 08:56 PM, Daniel D. Daugherty wrote: On 11/24/17 6:16 AM, Rahul Raghavan wrote: Hi, Please review the following fix proposal for 8191813. webrev - http://cr.openjdk.java.net/~rraghavan/8191813/webrev.00/ test/hotspot/jtreg/comp

Re: [10] RFR: 8191813: compiler/runtime/SpreadNullArg.java fails in tier1

2017-11-24 Thread mandy chung
Looks good.  Thanks for fixing the test, Rahul. Mandy On 11/24/17 3:16 AM, Rahul Raghavan wrote: Hi, Please review the following fix proposal for 8191813. webrev - http://cr.openjdk.java.net/~rraghavan/8191813/webrev.00/ jbs - https://bugs.openjdk.java.net/browse/JDK-8191813 -- reported is

Re: [10] RFR: 8191813: compiler/runtime/SpreadNullArg.java fails in tier1

2017-11-24 Thread Daniel D. Daugherty
On 11/24/17 6:16 AM, Rahul Raghavan wrote: Hi, Please review the following fix proposal for 8191813. webrev - http://cr.openjdk.java.net/~rraghavan/8191813/webrev.00/ test/hotspot/jtreg/compiler/runtime/SpreadNullArg.java     No comments. So this style of call:     mh_spreadInvoker.invokeEx

[10] RFR: 8191813: compiler/runtime/SpreadNullArg.java fails in tier1

2017-11-24 Thread Rahul Raghavan
Hi, Please review the following fix proposal for 8191813. webrev - http://cr.openjdk.java.net/~rraghavan/8191813/webrev.00/ jbs - https://bugs.openjdk.java.net/browse/JDK-8191813 -- reported issue - After the latest sync from jdk/jdk the test SpreadNullArg started to fail in tier1 on all pla