=?utf-8?q?Félix-Antoine?= Constantin
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/111...@github.com>


5chmidti wrote:

I think the regex solution might be brittle/not clean from a user perspective, 
with potential for a small amount of maintenance. Instead, I would also suggest 
a flag to error, warn, or silence these, defaulting to a warning.

Using `compile_commands.json` from a build configured for another compiler 
should provide a good experience.

- problem: `--flag` is not known to Clang, but would enable some part through a 
macro define -> missed analysis of important part of the code base?
- fine: `--other` is not known to Clang, but also not important to analysis 
because it does something during code-gen, etc.

The `--flag` example is why I'd be a bit worried about fully silencing these 
unknown flags, instead of a downgrade to a warning.

https://github.com/llvm/llvm-project/pull/111453
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to