<p dir="ltr"><br>
On Mar 3, 2013 7:39 AM, "David Chisnall" <<a href="mailto:csdavec@swan.ac.uk">csdavec@swan.ac.uk</a>> wrote:<br>
><br>
> Author: theraven<br>
> Date: Sun Mar  3 09:36:10 2013<br>
> New Revision: 176419<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=176419&view=rev">http://llvm.org/viewvc/llvm-project?rev=176419&view=rev</a><br>
> Log:<br>
> Default to enabling default-synthesized ivars on all platforms</p>
<p dir="ltr">That you could change this without causing any failures seems to indicate there's some missing test coverage I would think. Could you add a test?</p>
<p dir="ltr">><br>
> Modified:<br>
>     cfe/trunk/include/clang/Driver/ToolChain.h<br>
>     cfe/trunk/lib/Driver/ToolChains.h<br>
><br>
> Modified: cfe/trunk/include/clang/Driver/ToolChain.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/ToolChain.h?rev=176419&r1=176418&r2=176419&view=diff">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/ToolChain.h?rev=176419&r1=176418&r2=176419&view=diff</a><br>

> ==============================================================================<br>
> --- cfe/trunk/include/clang/Driver/ToolChain.h (original)<br>
> +++ cfe/trunk/include/clang/Driver/ToolChain.h Sun Mar  3 09:36:10 2013<br>
> @@ -147,7 +147,7 @@ public:<br>
><br>
>    /// IsObjCDefaultSynthPropertiesDefault - Does this tool chain enable<br>
>    /// -fobjc-default-synthesize-properties by default.<br>
> -  virtual bool IsObjCDefaultSynthPropertiesDefault() const { return false; }<br>
> +  virtual bool IsObjCDefaultSynthPropertiesDefault() const { return true; }<br>
><br>
>    /// IsEncodeExtendedBlockSignatureDefault - Does this tool chain enable<br>
>    /// -fencode-extended-block-signature by default.<br>
><br>
> Modified: cfe/trunk/lib/Driver/ToolChains.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.h?rev=176419&r1=176418&r2=176419&view=diff">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.h?rev=176419&r1=176418&r2=176419&view=diff</a><br>

> ==============================================================================<br>
> --- cfe/trunk/lib/Driver/ToolChains.h (original)<br>
> +++ cfe/trunk/lib/Driver/ToolChains.h Sun Mar  3 09:36:10 2013<br>
> @@ -297,10 +297,6 @@ public:<br>
>      return false;<br>
>    }<br>
><br>
> -  virtual bool IsObjCDefaultSynthPropertiesDefault() const {<br>
> -    return true;<br>
> -  }<br>
> -<br>
>    virtual bool IsEncodeExtendedBlockSignatureDefault() const {<br>
>      return true;<br>
>    }<br>
> @@ -564,10 +560,6 @@ public:<br>
>    virtual Tool &SelectTool(const Compilation &C, const JobAction &JA,<br>
>                             const ActionList &Inputs) const;<br>
><br>
> -  virtual bool IsObjCDefaultSynthPropertiesDefault() const {<br>
> -    return true;<br>
> -  }<br>
> -<br>
>    virtual bool IsIntegratedAssemblerDefault() const;<br>
>    virtual bool IsUnwindTablesDefault() const;<br>
>    virtual bool isPICDefault() const;<br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</p>