aco added a comment.

Hi, first of all I want to say that the name "HeaderFilterRegex" is a bit 
ambiguous, we don't know intuitively if we define what is filtered or what 
passes.

in my humble opinion alexfh's proposal would be much easier to use than 2 regex.

In D34654#3022728 <https://reviews.llvm.org/D34654#3022728>, @alexfh wrote:

> would a header glob list (similar to how the format of the `-checks=` flag) 
> be enough for the use cases folks have?



  For me yes it's enough, and will be welcome. 


Repository:
  rL LLVM

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

https://reviews.llvm.org/D34654

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D34654: Allow ... Alexander Kornienko via Phabricator via cfe-commits
    • [PATCH] D34654: A... Alan via Phabricator via cfe-commits

Reply via email to