[PATCH] D92743: [ScalarizeMaskedMemIntrin] Add new PM support

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 6 20:17:41 PST 2020


skatkov added a comment.

Please consider separate renaming to Legacy to NFC patch.



================
Comment at: llvm/lib/Transforms/Scalar/ScalarizeMaskedMemIntrin.cpp:849
+  auto &TTI = getAnalysis<TargetTransformInfoWrapperPass>().getTTI(F);
+  auto &DL = F.getParent()->getDataLayout();
+  return runImpl(F, TTI, DL);
----------------
You can get DL inside runImpl as it works identical in both legacy and new pass manager and does not depend on analysis.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92743



More information about the llvm-commits mailing list