[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