[clang-tools-extra] [clang-tidy] add default error message for performance-avoid-endl (PR #107867)

Congcong Cai via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 10 19:50:41 PDT 2024


================
@@ -225,3 +225,14 @@ void bad_custom_stream() {
   // CHECK-MESSAGES: :[[@LINE-1]]:13: warning: do not use 'std::endl' with streams; use '\n' instead [performance-avoid-endl]
   // CHECK-FIXES: logger << '\n';
 }
+
+namespace gh107859 {
+
+#define ENDL std::endl;
+
+void bad_macro() {
+  std::cout << ENDL;
----------------
HerrCai0907 wrote:

I am also confused. since I don't find auto fixes even replacement is emitted.
Maybe it is due to invalid location.
Do you think we should explicit avoid this replacement?

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


More information about the cfe-commits mailing list