[PATCH] D45711: [WebAssembly] Add an assertion for an invalid CFG

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 16 18:20:46 PDT 2018


dblaikie added a comment.

So does this make the other change NFC (under this assertion)? (ie: if the assertion is true, is "MBB" and "*Header" equivalent/the same thing?)

We can keep the other change even if this is true, if it's a nicer way of expressing the code. But I'm trying to ensure that there's not a semantic change hidden in here that would need testing. If all the things with semantic changes would trigger the assertion first/instead, then that's fine.


Repository:
  rL LLVM

https://reviews.llvm.org/D45711





More information about the llvm-commits mailing list