sammccall added a comment.

@malaperle to expand on what Eric said, adding proto hacks with false positives 
and no way to turn them off is indeed not the way to go here!
There's probably going to be other places we want to filter symbols too, and it 
should probably be extensible/customizable in some way.
We don't yet have enough examples to know what the structure should be 
(something regex based, a code-plugin system based on `Registry`, or something 
in between), thus the simplest/least invasive option for now (it's important 
for our internal rollout to have *some* mitigation in place).

@ioeric can you add a comment near the proto-filtering stuff indicating we 
should work out how to make this extensible?


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D46751



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

Reply via email to