[PATCH 1/4] Fixed compilation warning.

Jean-Charles BERTIN jc.bertin at axinoe.com
Wed Feb 27 19:36:45 PST 2013


You may ignore this patch because I've just learned that:
"adding a default: to a switch statement where the switch already covers
all of the cases: LLVM intentionally doesn't do this because doing so
silences a compiler warning when you add a new value to the
enumeration."

On Wed, 2013-02-27 at 19:56 -0500, Rafael EspĂ­ndola wrote:
> Which warning? Is the switch covering all cases? If so you should add
> an llvm_unreachable at the end instead.
> 
> On 27 February 2013 11:52, Jean-Charles BERTIN <jc.bertin at axinoe.com> wrote:
> > ---
> >  lib/CodeGen/CGObjCGNU.cpp | 1 +
> >  1 file changed, 1 insertion(+)
> >
> >
> > _______________________________________________
> > cfe-commits mailing list
> > cfe-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> >

-- 
Jean-Charles BERTIN
Axinoe - Software Engineer
Tel.: (+33) (0)1.80.82.59.23
Fax : (+33) (0)1.80.82.59.29
Skype: jcbertin
Web: <http://www.axinoe.com/>
Certificate Authority: <https://ca.axinoe.com/axinoe-root.crt>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3170 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130228/6526a075/attachment.bin>


More information about the cfe-commits mailing list