[llvm-commits] Removing redundant default cases

Chris Lattner clattner at apple.com
Mon Dec 19 17:11:43 PST 2011


Hi David,

Can you repost the patch?  The general idea of the patch seems sound.

-Chris

On Dec 14, 2011, at 12:05 PM, David Blaikie wrote:

> Bump - any interest? (this may be a bit outdated, I can resurrect &
> update it if desired)
> 
> On Fri, Sep 23, 2011 at 7:22 PM, David Blaikie <dblaikie at gmail.com> wrote:
>> A similar patch for one I've just sent out for clang - removing any default
>> cases in a switch that already has all the enum values specified (only for
>> switches over enums).
>> 
>> I don't know whether this stylistic/coding convention is consistent between
>> llvm and clang, so thought I'd ask first. Ideally once we get a good warning
>> for this in clang we could turn it on by default.
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list