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

[Chris Lattner]

Applied in r110370, thanks!


On Aug 4, 2010, at 6:24 PM, Cameron Esfahani wrote:

> 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
> 
> 
> 
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits