[PATCH] D87865: PR47468: Fix findPHICopyInsertPoint, so that copies aren't incorrectly inserted after an INLINEASM_BR.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 17 16:24:50 PDT 2020


efriedma added inline comments.


================
Comment at: llvm/lib/CodeGen/PHIEliminationUtils.cpp:57
+    if ((EHPadSuccessor && I->isCall()) ||
+        I->getOpcode() == TargetOpcode::INLINEASM_BR) {
+      InsertPoint = I.getReverse();
----------------
Is it possible to have multiple INLINEASM_BR instructions in the same block?  If it is, we need to find the first one, not the last one.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87865



More information about the llvm-commits mailing list