[cfe-commits] Building EFI with Clang on Mac OS X

[Douglas Gregor]

On Jan 31, 2011, at 2:13 PM, Carl Norum wrote:

> 
> On Jan 31, 2011, at 2:12 PM, Carl Norum wrote:
>> Thanks for going over these patches for me.  I've attached two new ones, split up as you suggested.  My comments follow:
> 
> ... except I didn't actually attach them.  Second attempt!

Mach-O patch looks good once the LLVM patches are committed.

Index: lib/Serialization/ASTReader.cpp
===================================================================
--- lib/Serialization/ASTReader.cpp	(revision 124313)
+++ lib/Serialization/ASTReader.cpp	(working copy)
@@ -96,6 +96,7 @@
   PARSE_LANGOPT_IMPORTANT(AltiVec, diag::warn_pch_altivec);
   PARSE_LANGOPT_IMPORTANT(Exceptions, diag::warn_pch_exceptions);
   PARSE_LANGOPT_IMPORTANT(SjLjExceptions, diag::warn_pch_sjlj_exceptions);
+  PARSE_LANGOPT_BENIGN(MSBitfields);

The MSBitfields bit should be "important", not "benign", since having -mms-bitfields when creating a PCH file but not when using that PCH file would be rather catastrophic. Otherwise, looks good!

	- Doug