[PATCH] Add ProgramFiles(x86) environment variable to defaults, fix some jaotc tests after graal sync

[Andrew Luo]

Just checking to see if there's any interest/comments on this.

Thanks,

-Andrew

-----Original Message-----
From: jtreg-dev <jtreg-dev-bounces at openjdk.java.net> On Behalf Of Andrew Luo
Sent: Monday, March 18, 2019 12:35 PM
To: jtreg-dev at openjdk.java.net
Subject: [PATCH] Add ProgramFiles(x86) environment variable to defaults, fix some jaotc tests after graal sync

Recently we added auto-detection of VS2017+ linkers to graal's jaotc (see https://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2019-January/032282.html), which uses the ProgramFiles(x86) environment variable to find VS2017+ linkers.  This change was included in the latest sync, however it doesn't work on WSL due to the ProgramFiles(x86) environment not being propagated from WSL to Win32.  I have a patch attached that fixes this (and tested by running tier 1 tests on WSL).

Feedback is welcome as always.

Thanks,

-Andrew