[PATCH] D21454: [PM] Port MergedLoadStoreMotion to the new pass manager, take two.

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 17 10:51:45 PDT 2016


davide updated this revision to Diff 61112.
davide added a comment.

Make `Impl` local to runOnFunction()/run() as suggested by David/Sean.
I was worried about an increased malloc() activity so I measured but I wasn't able to discover any substantial difference. 
We can re-evaluate in the future, if needed.
Also, Daniel needs to sign-off this patch before it can go in.


http://reviews.llvm.org/D21454

Files:
  include/llvm/InitializePasses.h
  include/llvm/Transforms/Scalar/MergedLoadStoreMotion.h
  lib/LTO/LTOCodeGenerator.cpp
  lib/Passes/PassBuilder.cpp
  lib/Passes/PassRegistry.def
  lib/Transforms/Scalar/MergedLoadStoreMotion.cpp
  lib/Transforms/Scalar/Scalar.cpp
  test/Transforms/InstMerge/exceptions.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21454.61112.patch
Type: text/x-patch
Size: 11489 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160617/a100fd0f/attachment.bin>


More information about the llvm-commits mailing list