================
@@ -477,6 +486,136 @@ void DiagnosticsEngine::setSeverityForAll(diag::Flavor 
Flavor,
       setSeverity(Diag, Map, Loc);
 }
 
+namespace {
+// FIXME: We should isolate the parser from SpecialCaseList and just use it
+// here.
+class WarningsSpecialCaseList : public llvm::SpecialCaseList {
----------------
kadircet wrote:

As you well know these discrepancies are coming from 
https://github.com/llvm/llvm-project/blob/main/clang/include/clang/Basic/LLVM.h.
 It's really hard to know/remember what's available there and what isn't. 
Moreover the header only forward-declares, so at usage side one actually needs 
to include the actual header from `llvm/...`.

Hence can't do this for `SpecialCaseList` but doing it for others that I can. 
But I think in the long run it'd be interesting to see if people still find 
this intermediate header useful or not.

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

Reply via email to