[PATCH] D81889: [gicombiner] Allow disable-rule option to disable all-except-...

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 16 10:27:08 PDT 2020


dsanders added a comment.

In D81889#2094543 <https://reviews.llvm.org/D81889#2094543>, @arsenm wrote:

> It's not clear to me what "'!foo-bar' - Enable rules from 'foo' to (and including) 'bar'" means. These have some kind of ordering?


Yes, it's the order that they're listed in the Combiner declaration (after flattening into a list) so it's a depth-first walk of the original list. It's mostly supported because there's no reason to special case it such that rule names don't work in the range syntax.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81889/new/

https://reviews.llvm.org/D81889





More information about the llvm-commits mailing list