[PATCH] D18576: Initial implementation of optimization bisect

[Andy Kaylor via llvm-commits]

andrew.w.kaylor added a comment.

One difference I see is that "optnone" is an attribute within the IR and so the pass manager shouldn't be looking for it (maybe that's the "trickery" that Chandler didn't want the pass manager to be in on?) whereas the OptBisect class exists for the sole purpose of deciding to skip optimizations, and so it necessarily (maybe) needs to be aware that there are some passes that can't be skipped.

The obvious alternative is to not have the pass manager involved in any of this and put the shouldRunPass() check directly into the passes themselves.  I was trying to avoid having to change a lot of different places to get this working, but that feels a bit more natural and it matches what's going to have to happen to skip individual optimizations.


Repository:
  rL LLVM

http://reviews.llvm.org/D18576