[PATCH] D133730: [SimplifyCFG] Don't hoist allocas
Momchil Velikov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 13 02:00:01 PDT 2022
chill accepted this revision.
chill added a comment.
This revision is now accepted and ready to land.
Thanks! LGTM.
There's a comment about `mayHaveSideEffects` that's not entirely correct, maybe it can be reworded somewhat?
https://github.com/llvm/llvm-project/blob/1f7b94e0ec7fae446e2ef9fcc41968fc711fb2c7/llvm/include/llvm/IR/Instruction.h#L630
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133730/new/
https://reviews.llvm.org/D133730
More information about the llvm-commits
mailing list