On Tue, Apr 24, 2012 at 1:49 PM, Nico Weber <[email protected]> wrote: > Hi, > > the attached patch fixes both issues mentioned in > http://llvm.org/PR12146 . Like suggested by Eli in that bug, it does > this by defining NULL to 0 instead of __null on windows. I think it > would be nice to have a __clang_null, but this patch fixes the > immediate issues I'm seeing without me having to argue with Eli :-) > Maybe I'll do that in a follow-up patch. > > Ok?
Not exactly sure what you did to make the test show up twice in the patch... Are you sure we want to do this for __MINGW32__? I don't know of anyone having issues here with MinGW. Otherwise, the patch is fine. -Eli _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
