RFR JDK-8193479: test/hotspot/jtreg/compiler/codegen/Test6896617.java fails after JDK-8184947
xueming.shen at oracle.com
Thu Dec 14 02:05:22 UTC 2017
webrev has been updated to fix the test directly.
Assume I would need hotspot guys' help to review and push into hs repo?
On 12/13/17, 4:52 PM, Martin Buchholz wrote:
> It would add more confusion to a something that's already difficult to
> understand. It's OK as a temporary measure, but I don't think we want
> product code just to enable tests in hotspot. Can the hotspot folks
> modify their test, perhaps making this code part of the test?
> On Wed, Dec 13, 2017 at 4:01 PM, Xueming Shen <xueming.shen at oracle.com
> <mailto:xueming.shen at oracle.com>> wrote:
> Please help review the change for JDK-8193479
> issue: https://bugs.openjdk.java.net/browse/JDK-8193479
> webrev: http://cr.openjdk.java.net/~sherman/8193479/webrev
> The internal interface ArrayEn/Decoder for ISO8859_1 has been removed
> because it is no longer used by StringCoding class, but it appears
> it is
> being used by hotspot Test6896617.java to verify the SSE instructions
> on x86. The proposed change here is to simply restore the internal
> ArrayEn/Decoder for ISO8859_1 to avoid blocking hotspot testing
> for now.
More information about the core-libs-dev