[cfe-commits] r45209 - /cfe/trunk/Driver/RewriteTest.cpp

[Steve Naroff]

Author: snaroff
Date: Wed Dec 19 13:16:49 2007
New Revision: 45209

URL: http://llvm.org/viewvc/llvm-project?rev=45209&view=rev
Log:

Soften the macro rewrite error to a warning (and change the text).


Modified:
    cfe/trunk/Driver/RewriteTest.cpp

Modified: cfe/trunk/Driver/RewriteTest.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/Driver/RewriteTest.cpp?rev=45209&r1=45208&r2=45209&view=diff

==============================================================================
--- cfe/trunk/Driver/RewriteTest.cpp (original)
+++ cfe/trunk/Driver/RewriteTest.cpp Wed Dec 19 13:16:49 2007
@@ -646,12 +646,10 @@
                                        IV->getLocation());
     if (Rewrite.ReplaceStmt(IV, Replacement)) {
       // replacement failed.
-      unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Error, 
-                     "rewriter could not replace sub-expression due to macros");
+      unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Warning, 
+                     "rewriting sub-expression within a macro (may not be correct)");
       SourceRange Range = IV->getSourceRange();
       Diags.Report(Context->getFullLoc(IV->getLocation()), DiagID, 0, 0, &Range, 1);
-      delete IV;
-      return Replacement;
     }
     delete IV;
     return Replacement;
@@ -670,12 +668,10 @@
           ParenExpr *PE = new ParenExpr(SourceLocation(), SourceLocation(), castExpr);
           if (Rewrite.ReplaceStmt(IV->getBase(), PE)) {
             // replacement failed.
-            unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Error, 
-                           "rewriter could not replace sub-expression due to macros");
+            unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Warning, 
+                           "rewriting sub-expression within a macro (may not be correct)");
             SourceRange Range = IV->getBase()->getSourceRange();
             Diags.Report(Context->getFullLoc(IV->getBase()->getLocStart()), DiagID, 0, 0, &Range, 1);
-            delete IV->getBase();
-            return PE;
           }
           delete IV->getBase();
           return PE;
@@ -935,12 +931,10 @@
                                         SourceLocation(), SourceLocation());
   if (Rewrite.ReplaceStmt(Exp, Replacement)) {
     // replacement failed.
-    unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Error, 
-                     "rewriter could not replace sub-expression due to macros");
+    unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Warning, 
+                     "rewriting sub-expression within a macro (may not be correct)");
     SourceRange Range = Exp->getSourceRange();
     Diags.Report(Context->getFullLoc(Exp->getAtLoc()), DiagID, 0, 0, &Range, 1);
-    delete Replacement;
-    return Exp;
   }
   
   // Replace this subexpr in the parent.
@@ -961,12 +955,10 @@
                                                  &SelExprs[0], SelExprs.size());
   if (Rewrite.ReplaceStmt(Exp, SelExp)) {
     // replacement failed.
-    unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Error, 
-                     "rewriter could not replace sub-expression due to macros");
+    unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Warning, 
+                     "rewriting sub-expression within a macro (may not be correct)");
     SourceRange Range = Exp->getSourceRange();
     Diags.Report(Context->getFullLoc(Exp->getAtLoc()), DiagID, 0, 0, &Range, 1);
-    delete SelExp;
-    return Exp;
   }
   delete Exp;
   return SelExp;
@@ -1295,12 +1287,10 @@
   CastExpr *cast = new CastExpr(Exp->getType(), call, SourceLocation());
   if (Rewrite.ReplaceStmt(Exp, cast)) {
     // replacement failed.
-    unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Error, 
-                     "rewriter could not replace sub-expression due to macros");
+    unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Warning, 
+                     "rewriting sub-expression within a macro (may not be correct)");
     SourceRange Range = Exp->getSourceRange();
     Diags.Report(Context->getFullLoc(Exp->getAtLoc()), DiagID, 0, 0, &Range, 1);
-    delete cast;
-    return Exp;
   }
   delete Exp;
   return cast;
@@ -1668,12 +1658,10 @@
     // Now do the actual rewrite.
     if (Rewrite.ReplaceStmt(Exp, PE)) {
       // replacement failed.
-      unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Error, 
-                       "rewriter could not replace sub-expression due to macros");
+      unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Warning, 
+                       "rewriting sub-expression within a macro (may not be correct)");
       SourceRange Range = Exp->getSourceRange();
       Diags.Report(Context->getFullLoc(Exp->getLocStart()), DiagID, 0, 0, &Range, 1);
-      delete PE;
-      return Exp;
     }
     delete Exp;
     return PE;
@@ -1681,12 +1669,10 @@
   // Now do the actual rewrite.
   if (Rewrite.ReplaceStmt(Exp, CE)) {
     // replacement failed.
-    unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Error, 
-                     "rewriter could not replace sub-expression due to macros");
+    unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Warning, 
+                     "rewriting sub-expression within a macro (may not be correct)");
     SourceRange Range = Exp->getSourceRange();
     Diags.Report(Context->getFullLoc(Exp->getLocStart()), DiagID, 0, 0, &Range, 1);
-    delete CE;
-    return Exp;
   }
   
   delete Exp;
@@ -1710,12 +1696,10 @@
                                                     ProtoExprs.size());
   if (Rewrite.ReplaceStmt(Exp, ProtoExp)) {
     // replacement failed.
-    unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Error, 
-                     "rewriter could not replace sub-expression due to macros");
+    unsigned DiagID = Diags.getCustomDiagID(Diagnostic::Warning, 
+                     "rewriting sub-expression within a macro (may not be correct)");
     SourceRange Range = Exp->getSourceRange();
     Diags.Report(Context->getFullLoc(Exp->getAtLoc()), DiagID, 0, 0, &Range, 1);
-    delete ProtoExp;
-    return Exp;
   }
   delete Exp;
   return ProtoExp;