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.s...@oracle.com> wrote: > Hi > > 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 > interface > ArrayEn/Decoder for ISO8859_1 to avoid blocking hotspot testing for now. > > thanks, > Sherman > > >
