[PATCH] D108837: [SimplifyCFG] Ignore free instructions when computing cost for folding branch to common dest
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 27 14:16:32 PDT 2021
lebedev.ri added inline comments.
================
Comment at: llvm/lib/Transforms/Utils/SimplifyCFG.cpp:3230
// Ignore dbg intrinsics, and the terminator.
if (isa<DbgInfoIntrinsic>(I) || isa<BranchInst>(I))
continue;
----------------
aeubanks wrote:
> xbolva00 wrote:
> > Skip also AssumeInst?
> should be handled with the new code
> and actually, same with DbgInfoIntrinsic, which I've updated
Surely assumes aren't speculatable?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108837/new/
https://reviews.llvm.org/D108837
More information about the llvm-commits
mailing list