Do not show macro expansion in strncat warnings, which can be defined as

a builtin.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@149657 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Sema/SemaChecking.cpp b/lib/Sema/SemaChecking.cpp
index 9dbee1b2b3c690772c150ee886374e7cccba73a4..10cf173216583105939cadbf5137c5dc8e776862 100644 (file)
--- a/lib/Sema/SemaChecking.cpp
+++ b/lib/Sema/SemaChecking.cpp
@@ -2742,12 +2742,22 @@ void Sema::CheckStrncatArguments(const CallExpr *CE,
   if (PatternType == 0)
     return;
 
+  // Generate the diagnostic.
+  SourceLocation SL = LenArg->getLocStart();
+  SourceRange SR = LenArg->getSourceRange();
+  SourceManager &SM  = PP.getSourceManager();
+
+  // If the function is defined as a builtin macro, do not show macro expansion.
+  if (SM.isMacroArgExpansion(SL)) {
+    SL = SM.getSpellingLoc(SL);
+    SR = SourceRange(SM.getSpellingLoc(SR.getBegin()),
+                     SM.getSpellingLoc(SR.getEnd()));
+  }
+
   if (PatternType == 1)
-    Diag(DstArg->getLocStart(), diag::warn_strncat_large_size)
-      << LenArg->getSourceRange();
+    Diag(SL, diag::warn_strncat_large_size) << SR;
   else
-    Diag(DstArg->getLocStart(), diag::warn_strncat_src_size)
-      << LenArg->getSourceRange();
+    Diag(SL, diag::warn_strncat_src_size) << SR;
 
   // Output a FIXIT hint if the destination is an array (rather than a
   // pointer to an array).  This could be enhanced to handle some
@@ -2773,9 +2783,8 @@ void Sema::CheckStrncatArguments(const CallExpr *CE,
   DstArg->printPretty(OS, Context, 0, getPrintingPolicy());
   OS << ") - 1";
 
-  Diag(LenArg->getLocStart(), diag::note_strncat_wrong_size)
-    << FixItHint::CreateReplacement(LenArg->getSourceRange(),
-                                    OS.str());
+  Diag(SL, diag::note_strncat_wrong_size)
+    << FixItHint::CreateReplacement(SR, OS.str());
 }
 
 //===--- CHECK: Return Address of Stack Variable --------------------------===//

diff --git a/test/Sema/warn-strncat-size.c b/test/Sema/warn-strncat-size.c
index 4233f25d5a4543f412c243938a9942e66272900f..7157edffea47b49a33653f4706089bd028008986 100644 (file)
--- a/test/Sema/warn-strncat-size.c
+++ b/test/Sema/warn-strncat-size.c
@@ -1,9 +1,20 @@
 // RUN: %clang_cc1 -Wstrncat-size -verify -fsyntax-only %s
+// RUN: %clang_cc1 -DUSE_BUILTINS -Wstrncat-size -verify -fsyntax-only %s
+// RUN: %clang_cc1 -fsyntax-only -Wstrncat-size -fixit -x c %s
+// RUN: %clang_cc1 -DUSE_BUILTINS -fsyntax-only -Wstrncat-size -fixit -x c %s
 
 typedef __SIZE_TYPE__ size_t;
-char  *strncat(char *, const char *, size_t);
 size_t strlen (const char *s);
 
+#ifdef USE_BUILTINS
+# define BUILTIN(f) __builtin_ ## f
+#else
+# define BUILTIN(f) f
+#endif
+
+#define strncat BUILTIN(strncat)
+char *strncat(char *restrict s1, const char *restrict s2, size_t n);
+
 struct {
   char f1[100];
   char f2[100][3];