[clang] [NFC] clang add DiagnosticsTestHelper decl (PR #112820)

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 18 06:48:37 PDT 2024


================
@@ -1024,6 +1024,9 @@ class DiagnosticsEngine : public RefCountedBase<DiagnosticsEngine> {
   /// @}
 };
 
+/// Declare the friend function for testing to avoid GCC warning.
----------------
erichkeane wrote:

couldn't we just put this at the top of the unittest file where this is defined/used?  Just do:

`namespace clang {
<declaration>
}`

It seems silly to do this for a function that is only used in the unittests.

https://github.com/llvm/llvm-project/pull/112820


More information about the cfe-commits mailing list