[llvm] r194486 - Revert "Remove unused variable."

Rafael EspĂ­ndola rafael.espindola at gmail.com
Tue Nov 12 08:54:49 PST 2013


On 12 November 2013 11:47, David Blaikie <dblaikie at gmail.com> wrote:
> Not necessarily possible/practical, since the macros could refer to
> different names based on the context in which they're used, etc (and there's
> only so much we can do to parse uninstantiated code)

I was thinking of something like "don't warn if the variable is
defined inside a macro".

> Usually for cases where we have a debug variable that's used in asserts but
> not otherwise, we either #ifndef NDEBUG the variable, or add a "(void)var;"
> near the assert.

Cheers,
Rafael



More information about the llvm-commits mailing list