[PATCH] D45150: Less conservative LoopSafetyInfo for headers

Piotr Padlewski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 5 09:25:32 PDT 2018


Prazek marked an inline comment as done.
Prazek added inline comments.


================
Comment at: include/llvm/Analysis/MustExecute.h:45
+  /// will be executed when the loop executes.
+  SmallPtrSet<const Instruction*, 32> GuaranteeToExecuteHeaderInstructions;
   // Used to update funclet bundle operands.
----------------
efriedma wrote:
> `AssertingVH<Instruction>`, to catch any invalidation bugs.
That is excellent comment, I haven't thought about possibility of moving and deleting the instructions from header.


Repository:
  rL LLVM

https://reviews.llvm.org/D45150





More information about the llvm-commits mailing list