[PATCH] D39084: [PM] Port BoundsChecking to the new PM.
Davide Italiano via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 13 17:32:01 PST 2017
davide added inline comments.
================
Comment at: llvm/include/llvm/Transforms/Instrumentation/BoundsChecking.h:27
+// checking on loads, stores, and other memory intrinsics.
+FunctionPass *createBoundsCheckingLegacyPass();
+
----------------
chandlerc wrote:
> davide wrote:
> > We generally don't put the old `create*` API in the header, is there a particular reason why this is here?
> I've been moving the create methods to the same header as the class. Otherwise, we end up in the awkward place where the .cpp file defines interfaces declared in two unrelated headers (MyPass.h and Scalar.h or whichever umbrella header had the createFoo.* thing originally).
>
> We didn't do this very carefully though and so probably need to go and move a bunch more of these declarations to better header files.
>
> (I chatted with Davide on IRC and this plan made sense to him...)
Yes, thanks!
https://reviews.llvm.org/D39084
More information about the llvm-commits
mailing list