[PATCH] D136752: Take memset_inline into account in analyzeLoadFromClobberingMemInst

[Clement Courbet via Phabricator via llvm-commits]

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


================
Comment at: llvm/lib/Transforms/Utils/VNCoercion.cpp:359
   // of the memset..
-  if (MI->getIntrinsicID() == Intrinsic::memset) {
+  if (isa<MemSetInst>(MI)) {
     if (DL.isNonIntegralPointerType(LoadTy->getScalarType())) {
----------------
nit: `if (const auto* memset_inst = dyn_cast<MemSetInst>(MI))` to avoid the redundant cast below ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136752