← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH 4/4] Drop gcc optimizer bug workaround

 

This should depend on your platforms gcc default setting or spec file if
you use one.  I'm not sure how this done with clang.  On Debain testing,
-O2 is the default build optimization setting and -O3 on msys2/mingw32
and msys2/mingw64.

On 12/14/2015 10:32 AM, Bernhard Stegmaier wrote:
> I also assumed something like that and therefore had a look yesterday.
> On OSX it seems to default to -O3, at least that's what I have seen with
> "make VERBOSE=1".
> 
> 
> Regards,
> Bernhard
> 
> On 2015-12-14 16:01, Mark Roszko wrote:
>> Stupid question, doesn't this change remove -O2 entirely for all gcc
>> builds? Previously it defaulted to -O2 if the bug wasn't present. Now
>> it never optimizes as -O0 is the default.
>>
>> On Sat, Dec 12, 2015 at 2:59 PM, Wayne Stambaugh
>> <stambaughw@xxxxxxxxx> wrote:
>>> I committed your patch in the product branch r6370.  I tested it on
>>> msys1/mingw32, msys2/mingw32, msys2/mingw64, and Debian testing without
>>> issues.  You should also be able to be more aggressive with your
>>> optimization settings should you so desire.  Please keep in mind, if you
>>> do this and it breaks, you get keep both pieces.
>>>
>>> On 12/1/2015 2:14 AM, Simon Richter wrote:
>>>>
>>>> As boost::polygon is gone, we can drop the workaround for older gcc.
>>>> ---
>>>>  CMakeLists.txt | 15 ---------------
>>>>  1 file changed, 15 deletions(-)
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> Mailing list: https://launchpad.net/~kicad-developers
>>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>>> More help   : https://help.launchpad.net/ListHelp
>>>>
>>>
>>>
>>> _______________________________________________
>>> Mailing list: https://launchpad.net/~kicad-developers
>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>> More help   : https://help.launchpad.net/ListHelp


Follow ups

References