[cfe-commits] r70800 - in /cfe/trunk: lib/Sema/SemaExpr.cpp test/SemaCXX/offsetof.cpp

Eli Friedman eli.friedman at gmail.com
Sun May 3 14:22:18 PDT 2009


Author: efriedma
Date: Sun May  3 16:22:18 2009
New Revision: 70800

URL: http://llvm.org/viewvc/llvm-project?rev=70800&view=rev
Log:
Fix/re-enable test.


Modified:
    cfe/trunk/lib/Sema/SemaExpr.cpp
    cfe/trunk/test/SemaCXX/offsetof.cpp

Modified: cfe/trunk/lib/Sema/SemaExpr.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaExpr.cpp?rev=70800&r1=70799&r2=70800&view=diff

==============================================================================
--- cfe/trunk/lib/Sema/SemaExpr.cpp (original)
+++ cfe/trunk/lib/Sema/SemaExpr.cpp Sun May  3 16:22:18 2009
@@ -4734,7 +4734,7 @@
       << SourceRange(CompPtr[1].LocStart, CompPtr[NumComponents-1].LocEnd);
 
   if (!Dependent) {
-    bool DidWarnAboutNonPOD;
+    bool DidWarnAboutNonPOD = false;
     
     // FIXME: Dependent case loses a lot of information here. And probably
     // leaks like a sieve.

Modified: cfe/trunk/test/SemaCXX/offsetof.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/offsetof.cpp?rev=70800&r1=70799&r2=70800&view=diff

==============================================================================
--- cfe/trunk/test/SemaCXX/offsetof.cpp (original)
+++ cfe/trunk/test/SemaCXX/offsetof.cpp Sun May  3 16:22:18 2009
@@ -1,6 +1,4 @@
-// FIXME: This test fails on 32-bit platforms? Investigate.
-
-// RUNX: clang-cc -fsyntax-only -verify %s -Winvalid-offsetof
+// RUN: clang-cc -fsyntax-only -verify %s -Winvalid-offsetof
 
 struct NonPOD {
   virtual void f();





More information about the cfe-commits mailing list