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

[Nick Desaulniers via Phabricator via llvm-commits]

nickdesaulniers planned changes to this revision.
nickdesaulniers added a comment.

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.

I'll add `-verify-machineinstrs` to this test at least.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142924