[PATCH] D38768: Add remarks describing when a pass changes the IR instruction count of a module

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 24 13:47:16 PDT 2017


davide added a comment.

In https://reviews.llvm.org/D38768#903822, @paquette wrote:

> Now that there's discussion on making the new pass manager the default pass manager, I'm wondering if it'd make sense to just go ahead and implement this functionality there entirely? I was hoping to get it done with the LegacyPassManager first then port it over after.
>
> Also, the test is pretty big now to cover all of the pass manager types. Does anyone think there's a more succinct way to do this?


Sorry, I lost part of your question.
I'm not immediately concerned with the size of the test. If you want you can probably split in multiple files but I'm not sure  if that buys us something.


https://reviews.llvm.org/D38768





More information about the llvm-commits mailing list