[llvm] r182832 - Temporary fix to get rid of gcc warning.

Rafael EspĂ­ndola rafael.espindola at gmail.com
Wed May 29 11:44:02 PDT 2013


> Generally we prefer to simply disable or fix bad warnings rather than
> workaround them in code. (which usually amounts to disabling GCC warnings
> and fixing Clang warnings)
>
> Which warning was the problem here & can we just disable it? Does Clang have
> an equivalent (& better/correct/not-false-positive-on-this-code) warning?

I think it was a valid warning about the comparison always being true
and if remember the email correctly it actually found a bug on the
enum values they use. Vincent?

Cheers,
Rafael



More information about the llvm-commits mailing list