[cfe-commits] Patch to fix http://llvm.org/bugs/show_bug.cgi?id=7820

Cameron Esfahani dirty at apple.com
Wed Aug 4 18:24:05 PDT 2010


The 64-bit Visual Studio target info should not override the default X86_64 VA list declaration.

Index: tools/clang/lib/Basic/Targets.cpp
===================================================================
--- tools/clang/lib/Basic/Targets.cpp	(revision 110284)
+++ tools/clang/lib/Basic/Targets.cpp	(working copy)
@@ -1489,9 +1489,6 @@
     WindowsX86_64TargetInfo::getTargetDefines(Opts, Builder);
     Builder.defineMacro("_M_X64");
   }
-  virtual const char *getVAListDeclaration() const {
-    return "typedef char* va_list;";
-  }
 };
 } // end anonymous namespace

Cameron Esfahani
dirty at apple.com

"I cannot for the life of me understand why, while people without driver's licenses are not allowed on public roads, in bookstores one can find any number of books by persons without decency - let alone knowledge."

"His Master's Voice", Stanislaw Lem







More information about the cfe-commits mailing list