RFR(XS): 8232083: Minimal VM is broken after JDK-8231586

[Tom Rodriguez]

Sorry, I'd prepared this too quickly during a meeting.  I've corrected 
the code at http://cr.openjdk.java.net/~never/8232083.1/webrev and built 
and ran both a normal and minimal JVM using these changes.

tom

Jie Fu wrote on 10/14/19 5:56 PM:
> Hi Tom,
> 
> Thanks for fixing this.
> 
> The patch seems to break the vm build:
> ---------------------------------------------
> /home/fool/workspace/jdk-dev/src/hotspot/share/compiler/oopMap.cpp:343:3: error: 
> expected unqualified-id before '{' token
>     {
>     ^
> /home/fool/workspace/jdk-dev/src/hotspot/share/compiler/oopMap.cpp:394:1: error: 
> expected declaration before '}' token
>   }
>   ^
> lib/CompileJvm.gmk:176: recipe for target 
> '/home/fool/workspace/jdk-dev/build/linux-x86_64-server-release/hotspot/variant-server/libjvm/objs/oopMap.o' 
> failed
> ---------------------------------------------
> 
> Best regards,
> Jie
> 
> On 2019/10/15 上午4:34, Tom Rodriguez wrote:
>> Yes I mistranslated this code in the case where that ifndef triggers. 
>> I think this http://cr.openjdk.java.net/~never/8232083/webrev/ is a 
>> better translation of the original code.  Can I get a review of this 
>> and I will push it.
>>
>> tom
>>
>> Jie Fu wrote on 10/13/19 11:35 PM:
>>> Forward this to Tom. Thanks.
>>>
>>> On 2019/10/11 下午8:07, Jie Fu wrote:
>>>> Hi all,
>>>>
>>>> JDK-8232151 had been fixed.
>>>>
>>>> Could you please review this fix and give me some advice?
>>>>
>>>> JBS:    https://bugs.openjdk.java.net/browse/JDK-8232083
>>>> Webrev: http://cr.openjdk.java.net/~jiefu/8232083/webrev.00/
>>>>
>>>> Thanks a lot.
>>>> Best regards,
>>>> Jie
>>>>
>>>> On 2019/10/11 上午11:42, Jie Fu wrote:
>>>>> Please hold off reviewing this until JDK-8232151 
>>>>> <https://bugs.openjdk.java.net/browse/JDK-8232151> is fixed.
>>>>
>>>
>