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 <[email protected]> wrote: > > --- > > lib/CodeGen/CGObjCGNU.cpp | 1 + > > 1 file changed, 1 insertion(+) > > > > > > _______________________________________________ > > cfe-commits mailing list > > [email protected] > > 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>
smime.p7s
Description: S/MIME cryptographic signature
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
