RFR(XXS): 8200288: [SPARC] "assert(!(is_cti(prev) && is_cti(insn))) failed: CTI-CTI not allowed"

[Vladimir Kozlov]

I am fine with this fix.

Thanks,
Vladimir

On 5/24/18 12:31 PM, Nils Eliasson wrote:
> Hi Patric,
> 
> The link points to the wrong issue. Should be 
> https://bugs.openjdk.java.net/browse/JDK-8200288
> 
> The patch looks good.
> 
> // Nils
> 
> 
> On 2018-05-24 15:37, Patric Hedlin wrote:
>> Dear all,
>>
>> I would like to ask for help to review the following change/update:
>>
>> Issue: https://bugs.openjdk.java.net/browse/JDK-8200288
>>
>> 8200288: [SPARC] "assert(!(is_cti(prev) && is_cti(insn))) failed: 
>> CTI-CTI not allowed"
>>
>> diff -r 5715962c8831 -r 7a1ee2a4f6a6 
>> src/hotspot/cpu/sparc/assembler_sparc.hpp
>> --- a/src/hotspot/cpu/sparc/assembler_sparc.hpp    Thu May 24 14:35:32 
>> 2018 +0530
>> +++ b/src/hotspot/cpu/sparc/assembler_sparc.hpp    Tue Apr 24 13:59:02 
>> 2018 +0200
>> @@ -783,8 +783,10 @@
>>    void flush() {
>>  #ifdef VALIDATE_PIPELINE
>>      assert(_delay_state == NoDelay, "Ending code with a delay-slot.");
>> +#ifdef COMPILER2
>>      validate_no_pipeline_hazards();
>>  #endif
>> +#endif
>>      AbstractAssembler::flush();
>>    }
>>
>> Best regards,
>> Patric
>