[PATCH] D112382: [InstSimplify] Refactor invariant.group load folding

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks accepted this revision.
aeubanks added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/include/llvm/IR/Value.h:697
+  /// If \p AllowInvariant is true then this method also looks through
+  /// strip.invariant and launder.invariant intrinsics.
+  ///
----------------
nit: `strip.invariant.group` and `launder.invariant.group`


================
Comment at: llvm/include/llvm/IR/Value.h:714
       const DataLayout &DL, APInt &Offset, bool AllowNonInbounds,
+      bool AllowInvariant = false,
       function_ref<bool(Value &Value, APInt &Offset)> ExternalAnalysis =
----------------
`AllowInvariantGroup`


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112382/new/

https://reviews.llvm.org/D112382