[Enh]: 8230091: Add verification of clean_catch_blocks

[Vladimir Kozlov]

Hi Nils

Why you execute two times?:

      clean_catch_blocks(phase);
+    DEBUG_ONLY(clean_catch_blocks(phase, true /* verify */);)

Did you mean something like next?:

clean_catch_blocks(phase, DEBUG_ONLY(true) NOT_DEBUG(false) /* verify */);

Thanks,
Vladimir

On 8/23/19 1:30 AM, Nils Eliasson wrote:
> Hi,
> 
> This patch adds some verfication of clean_catch_blocks in zBarrierSetC2. This would have catched the bug i recently 
> fixed: JDK-8229970
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8230091
> 
> Webrev: http://cr.openjdk.java.net/~neliasso/8230091/webrev.01/
> 
> Regards,
> 
> Nils
>