[PATCH] D142924: [llvm][IfConversion] update successor list when merging INLINEASM_BR

Tue Feb 7 11:22:31 PST 2023

nickdesaulniers added a comment.

In D142924#4110734 <https://reviews.llvm.org/D142924#4110734>, @nickdesaulniers wrote:

> In D142924#4107194 <https://reviews.llvm.org/D142924#4107194>, @nickdesaulniers wrote:
>
>> (I'm going to add a machine verifier test for this first)
>
> Huh, the checks I added in https://reviews.llvm.org/D130290 catch this.  How come machine verifier doesn't run from clang? Is there a way to force it on from clang?
>
> Looks like `-mllvm -verify-machineinstrs` does it.

Also `-DLLVM_ENABLE_EXPENSIVE_CHECKS=ON` seems to enable `-verify-machineinstrs` by default based on a build report I just got that I think just tested the parent commit (a known failure with -verify-machineinstrs).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142924/new/

https://reviews.llvm.org/D142924