[PATCH] D44464: OptBisect is improved to be overridden in LLVMContext

Yevgeny Rouban via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 15 20:27:22 PDT 2018


yrouban added a comment.

In https://reviews.llvm.org/D44464#1039293, @fedor.sergeev wrote:

> Is it a matter of properly wording commit message?


I think that the class name OptBisect is a bit misleading. The word //Bisect// explains just one way of using the skip functionality. I would change to something like //OptPassGate//, meaning that the class just decides if the pass should be skipped or run.


https://reviews.llvm.org/D44464





More information about the llvm-commits mailing list