[patch] Remove -fobjc-default-synthesize-properties

Jordan Rose jordan_rose at apple.com
Fri Sep 27 11:49:02 PDT 2013

On Sep 27, 2013, at 11:33 , Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:

>> I see. If driver always passes -fobjc-default-synthesize-properties then our users will be OK.
>> But, we still need to use legacy property synthesis for testing purposes. But we can always use cc1 directly.
>> LGTM them.
> Sorry, it looks like there is a bit of confusion.
> the current status is:
> * Driver alway passes -fobjc-default-synthesize-properties
> * -cc1 defaults to false, but only 3 tests expect the behavior of not
> having -fobjc-default-synthesize-properties.
> * There is no -fno-objc-default-synthesize-properties, so the only way
> to get the off behavior is to run -cc1 directly.
> With the patch we get:
> * Driver doesn't pass -fobjc-default-synthesize-properties.
> * -cc1 always has behavior we currently get by passing
> -fobjc-default-synthesize-properties.
> * The 3 tests that were not using -fobjc-default-synthesize-properties
> are updated (super-class-protocol-conformance.m,
> unimplemented-protocol-prop.m, property-category-1.m)
> * -fobjc-default-synthesize-properties  is removed from -cc1 too, so
> there is no way to turn it off.

Why not just change the -cc1 option to -fno-...? Or even drop the -f, if it's just a cc1 option?

More information about the cfe-commits mailing list