baloghadamsoftware added a comment.

I am a bit afraid of the performance of the feature because of the extensive 
use of matchers. Based on my previous experiences matcher creation and 
disposition are expensive operations. Matchers are meant to be a kind of 
"static" structures created only once and destroyed at the end. Using them in 
the core infrastructure may negatively affect the execution time of the whole 
analysis. Maybe we should get rid of them before turning on this feature as 
default (which is the ultimate goal).


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

https://reviews.llvm.org/D63279



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to