[cfe-commits] Fix for the -mms-bitfields commandline parameter

Eli Friedman eli.friedman at gmail.com
Tue Oct 9 16:40:40 PDT 2012


On Tue, Oct 9, 2012 at 8:53 AM, Jeremiah Zanin
<Jeremiah.Zanin at volition-inc.com> wrote:
> Ok, here's the updated patch.

+/// isMsStrust - Get whether or not this is an ms_struct which can
+/// be turned on with an attribute, pragma, or -mms-bitfields
+/// commandline option.

/// isMsStruct - Get whether or not this record uses ms_struct layout.
/// This which can be turned on with an attribute, pragma, or the
/// -mms-bitfields command-line option.


There are some existing tests in test/Sema/pragma-ms_struct.c ; please
move the tests which don't depend on the -mms-bitfields flag there,
and add a test next to it to test -mms-bitfields.  The current
"test/CodeGen/mms-bitfields.c" is not a good example to follow.

-Eli



More information about the cfe-commits mailing list