From 81ab3cfcf2f312e82c51e58a1fce8e85e9fe59c3 Mon Sep 17 00:00:00 2001
From: Fariborz Jahanian <fjahanian@apple.com>
Date: Wed, 13 Apr 2011 20:31:26 +0000
Subject: [PATCH] No fixit hint for builtin expressions which are defined in a
 macro. // rdar://9091893

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@129465 91177308-0d34-0410-b5e6-96231b3b80d8
---
 lib/Sema/SemaExpr.cpp       | 14 ++++++++------
 test/FixIt/no-macro-fixit.c | 15 +++++++++++++++
 2 files changed, 23 insertions(+), 6 deletions(-)
 create mode 100644 test/FixIt/no-macro-fixit.c

diff --git a/lib/Sema/SemaExpr.cpp b/lib/Sema/SemaExpr.cpp
index 96b2e56b41..3d2b14933e 100644
--- a/lib/Sema/SemaExpr.cpp
+++ b/lib/Sema/SemaExpr.cpp
@@ -10070,9 +10070,6 @@ void Sema::DiagnoseAssignmentAsCondition(Expr *E) {
     return;
   }
 
-  SourceLocation Open = E->getSourceRange().getBegin();
-  SourceLocation Close = PP.getLocForEndOfToken(E->getSourceRange().getEnd());
-
   Diag(Loc, diagnostic) << E->getSourceRange();
 
   if (IsOrAssign)
@@ -10082,9 +10079,14 @@ void Sema::DiagnoseAssignmentAsCondition(Expr *E) {
     Diag(Loc, diag::note_condition_assign_to_comparison)
       << FixItHint::CreateReplacement(Loc, "==");
 
-  Diag(Loc, diag::note_condition_assign_silence)
-    << FixItHint::CreateInsertion(Open, "(")
-    << FixItHint::CreateInsertion(Close, ")");
+  SourceLocation Open = E->getSourceRange().getBegin();
+  SourceLocation Close = E->getSourceRange().getEnd();
+  if (!Open.isMacroID() && !Close.isMacroID()) {
+    SourceLocation LocForEndOfToken = PP.getLocForEndOfToken(Close);
+    Diag(Loc, diag::note_condition_assign_silence)
+      << FixItHint::CreateInsertion(Open, "(")
+      << FixItHint::CreateInsertion(LocForEndOfToken, ")");
+  }
 }
 
 /// \brief Redundant parentheses over an equality comparison can indicate
diff --git a/test/FixIt/no-macro-fixit.c b/test/FixIt/no-macro-fixit.c
new file mode 100644
index 0000000000..3685ab1087
--- /dev/null
+++ b/test/FixIt/no-macro-fixit.c
@@ -0,0 +1,15 @@
+// RUN: %clang_cc1 -pedantic -fixit -x c %s
+// rdar://9091893
+
+#define va_arg(ap, type)    __builtin_va_arg(ap, type)
+typedef __builtin_va_list va_list;
+
+void myFunc() {
+    va_list values;
+    
+    int value;
+
+    while (value = va_arg(values, int)) {  // expected-warning {{using the result of an assignment as a condition without parentheses}} \
+                                           // expected-note {{use '==' to turn this assignment into an equality comparison}}
+    }
+}
-- 
2.50.1