search

berkus / mclinker

Automatically exported from code.google.com/p/mclinker
Other
0 stars 0 forks source link

Ad --enable-assertions option in ./configure #100

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
So far, MCLinker adds all assertions. But if we add NDEBUG macro and remove all 
assertions, then we will see there are many warnings hiding in our code. (most 
of them are unused variable).

I will add an option in ./configure to turn off all assertions and start to 
remove all warnings.

Original issue reported on code.google.com by LubaTang on 27 Nov 2012 at 3:24

GoogleCodeExporter commented 9 years ago 
I think this is unnecessary since our project depends on llvm-config.

If llvm configuration with --disable-assertions, it will automatically add 
-DNDEBUG.
if not, then we still keep assertions.

I close this issue now, but, if any requirement, please re-open it.
Thanks!

Original comment by wenhan...@gmail.com on 25 Dec 2013 at 9:41