[PATCH] D21462: [PM] Make the the new pass manageg support fully generic extra arguments to run methods, both for transform passes and analysis passes.

Sean Silva via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 3 12:56:27 PDT 2016


silvas added a comment.

How do you plan to marshal the variadic arguments for analyses with a unified analysis manager for all IRUnit's (w/ dependency tracking)? That seems annoying.

I should have some code for the unified analysis manager up today hopefully.

I hear your argument (IIRC we discussed this at a social) about back pointers being somewhat annoying, but the ability to use just a `void*` to represent the information passed to an analysis makes it much easier to have a unified analysis manager.

Note that this issue doesn't apply to transformations since they don't participate in the analysis management.


https://reviews.llvm.org/D21462





More information about the llvm-commits mailing list