<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Nov 21, 2012 at 3:50 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Wed, Nov 21, 2012 at 3:46 PM, Matt Beaumont-Gay <<a href="mailto:matthewbg@google.com">matthewbg@google.com</a>> wrote:<br>

> On Wed, Nov 21, 2012 at 3:40 PM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com">chandlerc@gmail.com</a>> wrote:<br>
>> Modified: cfe/trunk/lib/Driver/ToolChains.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.cpp?rev=168473&r1=168472&r2=168473&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.cpp?rev=168473&r1=168472&r2=168473&view=diff</a><br>

>> ==============================================================================<br>
>> --- cfe/trunk/lib/Driver/ToolChains.cpp (original)<br>
>> +++ cfe/trunk/lib/Driver/ToolChains.cpp Wed Nov 21 17:40:23 2012<br>
>> @@ -2230,10 +2230,15 @@<br>
>>    return *T;<br>
>>  }<br>
>><br>
>> -void Linux::addClangTargetOptions(ArgStringList &CC1Args) const {<br>
>> +void Linux::addClangTargetOptions(const ArgList &DriverArgs,<br>
>> +                                  ArgStringList &CC1Args) const {<br>
>>    const Generic_GCC::GCCVersion &V = GCCInstallation.getVersion();<br>
>> -  if (V >= Generic_GCC::GCCVersion::Parse("4.7.0") ||<br>
>> -      getTriple().getEnvironment() == llvm::Triple::Android)<br>
>> +  bool UseInitArrayDefault<br>
>> +    = V >= Generic_GCC::GCCVersion::Parse("4.7.0") ||<br>
><br>
> Is the preferred style really to put the '=' on the next line like this?<br>
<br>
</div>It's a common style, I think trying to distinguish between an operator<br>
like == and the initializer for a variable...<br>
<br>
I don't actually have any preference here. Doug once told me to do it<br>
like this, and I've done so as consistently as I can since...<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div><br></div><div>Huh. I've always done it the other way.</div><div><br></div><div>-eric </div></div><br></div>