> On 2015-Dec-07, at 13:29, Bruno Cardoso Lopes <[email protected]> wrote:
>
> bruno added a comment.
>
> Hi Argyrios,
>
> Thanks for the suggestions, will apply them.
> The assertion seems important to catch subtle bugs, are you sure it should be
> placed inside a "#ifndef NDEBUG”?
FYI, the definition for `assert` is something like:
#ifdef NDEBUG
#define assert(cond)
#else
#define assert(cond) \
if (!(cond)) { assertion_failed(#FILE ":" #LINE ": " #cond); };
#endif
>
>
> http://reviews.llvm.org/D15173
>
>
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
