[PATCH] -Wassign-enum: compare unqualified types

David Blaikie dblaikie at gmail.com
Thu Dec 5 13:56:46 PST 2013


  Sure, seems reasonable. It's also consistent with the -Wconstant-conversion warning which warns about implicit conversions that truncate (char a = 500; char b = (char)500; const char c = (char)500; only warns about 'a').

  Perhaps an additional test case or two to demonstrate that the suppression works in similar contexts to the warning, even in the qualified match case. Maybe comments describing this in the test and/or source? Up to you there.

http://llvm-reviews.chandlerc.com/D2341



More information about the cfe-commits mailing list