[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