================
@@ -70,6 +76,16 @@ void clang::ProcessWarningOptions(DiagnosticsEngine &Diags,
   else
     Diags.setExtensionHandlingBehavior(diag::Severity::Ignored);
 
+  if (!Opts.DiagnosticSuppressionMappingsFile.empty()) {
+    if (auto Buf =
----------------
kadircet wrote:

i think 
[Buf](https://github.com/search?type=code&auto_enroll=true&q=repo%3Allvm%2Fllvm-project+%2Fbuf.*%3D.*%5Cn%3F.*getBufferForFile%2F)
 version is more common than anything involving 
[File](https://github.com/search?type=code&auto_enroll=true&q=repo%3Allvm%2Fllvm-project+%2Ffile.*%3D.*%5Cn%3F.*getBufferForFile%2F).

Moreover, this is really a memory-buffer object, that doesn't have ties to any 
file anymore. So I'd argue that conceptually calling it `Buf` also makes more 
sense.

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