[Coin-discuss] Makefile.CYGWIN

[Matthew Galati]

i copy and pasted the section on -O levels from linux make to cygwin make...

-DNDEBUG worked fine for me, and seems to run fine on my stuff
-DCOIN_ASSERT does not compile (see CoinError)

Matt

Matthew Galati - Optimization Developer
SAS Institute - Analytical Solutions
Phone 919-531-0332, R5327 
Fax   919-677-4444
http://coral.ie.lehigh.edu/~magh
http://ordlnx2.na.sas.com/projects/OptWiki
http://www.sas.com/technologies/analytics/optimization/




> On Tue, 5 Jul 2005, Matthew Galati wrote:
>
>> Hi Matt,
>>
>> No patch or test done by me.
>>
>> Just suggesting to add those def's to the build (as in Linux), since 
>> some COIN modules are heavy on assertions and it could be an 
>> unnecessary performance penalty under -O.
>
>
> It's a good suggestion, I'd just rather have somebody try the obvious 
> patch before I commit it, to make sure it doesn't break anything.  I 
> don't have a Cygwin installation handy ATM.  I wouldn't expect any 
> problems, but would prefer to be prudent.
>
>>
>> Matt
>>
>> Matthew Galati - Optimization Developer
>> SAS Institute - Analytical Solutions
>> Phone 919-531-0332, R5327 Fax   919-677-4444
>> http://coral.ie.lehigh.edu/~magh
>> http://ordlnx2.na.sas.com/projects/OptWiki
>> http://www.sas.com/technologies/analytics/optimization/
>>
>>
>>
>>
>>> On Mon, 4 Jul 2005, Matthew Galati wrote:
>>>
>>>> Is there any reason why Makefile.CYGWIN does not define -DNDEBUG 
>>>> and -DCOIN_ASSERT when OptLevel=-O, as in Makefile.Linux?
>>>
>>>
>>>
>>> I doubt it.  Probably somebody thought it would be a good idea for 
>>> Linux and didn't think about--or couldn't test--Cygwin.  I can't 
>>> tell from the log who made that particular change.
>>>
>>> Do you have a tested patch?
>>>
>>>>
>>>> Thanks,
>>>> Matt
>>>>
>>>>
>>>
>> _______________________________________________
>> Coin-discuss mailing list
>> Coin-discuss at list.coin-or.org
>> http://list.coin-or.org/mailman/listinfo/coin-discuss
>>
>