[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