[PATCH] Port ADCE to the new pass manager

Justin Bogner via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 25 15:46:54 PDT 2015


The attached patch ports ADCE to the new pass manager. It's basically
mechanical and based off of some of the ports you've already done so
it's probably good to go in, but I have a couple of questions about
parts that seemed a little odd.

1. The legacy pass states it preserves GlobalsAAWrapperPass, but the new
   pass returns PreservedAnalyses::none when it does work. This seems
   inconsistent, but maybe I'm missing something.

2. Similarly, some of the other passes you've ported do
   AU.setPreservesCFG(), but this doesn't seem to be reflected in the
   new passes. How's this going to work?

3. As per my reply to "Port SimplifyCFG to the new pass manager", it
   seems like we don't have good testing infrastructure in terms of
   preserved analyses. Do you have any ideas for how we're going to deal
   with that?

Anyway, let me know if this is good to land. I might tackle a couple
more of these in the near future, and some clarity on the above
questions would help.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: newpm-adce.patch
Type: text/x-patch
Size: 6840 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150925/87d38ac6/attachment.bin>


More information about the llvm-commits mailing list