similar to: RFC Enable -Wimplicit-fallthrough for clang as well as GCC

Hi llvmdev, llvm-commits, There was a discussion on this topic a while ago, and now I've decided to make a formal proposal and post it here. I propose to add the LLVM_FALLTHROUGH macro for specifying intended fall-through locations between switch cases. *INTRODUCTION* The switch construct of C/C++ languages allows fall-throughs between switch labels when control flow is not directed

I came across a weird-looking use of LLVM_FALLTHROUGH which I think is completely spurious, but I figured I should check with the group mind before ripping it out. Basically, if you have multiple cases with no code in between, you do *not* need LLVM_FALLTHROUGH, right? switch (Foo) { case Bar1: LLVM_FALLTHROUGH; // not needed case Bar2: some code; return; case Bar3:

Ping. On Fri, Jul 27, 2012 at 3:42 AM, Alexander Kornienko <alexfh at google.com>wrote: > On Fri, Jul 27, 2012 at 2:02 AM, Richard Smith <richard at metafoo.co.uk>wrote: > >> On Thu, Jul 26, 2012 at 4:24 PM, Jim Grosbach <grosbach at apple.com> wrote: >> >>> >>> On Jul 26, 2012, at 4:07 PM, Chris Lattner <clattner at apple.com> wrote:

On Fri, Jul 27, 2012 at 2:02 AM, Richard Smith <richard at metafoo.co.uk>wrote: > On Thu, Jul 26, 2012 at 4:24 PM, Jim Grosbach <grosbach at apple.com> wrote: > >> >> On Jul 26, 2012, at 4:07 PM, Chris Lattner <clattner at apple.com> wrote: >> >> > <dropping llvm-commits> >> > >> > On Jul 2, 2012, at 9:59 AM, Alexander

Ping. On Thu, Aug 9, 2012 at 3:04 PM, Alexander Kornienko <alexfh at google.com>wrote: > Ping. > > > On Fri, Jul 27, 2012 at 3:42 AM, Alexander Kornienko <alexfh at google.com>wrote: > >> On Fri, Jul 27, 2012 at 2:02 AM, Richard Smith <richard at metafoo.co.uk>wrote: >> >>> On Thu, Jul 26, 2012 at 4:24 PM, Jim Grosbach <grosbach at

On Jul 26, 2012, at 4:07 PM, Chris Lattner <clattner at apple.com> wrote: > <dropping llvm-commits> > > On Jul 2, 2012, at 9:59 AM, Alexander Kornienko wrote: > >> Hi llvmdev, llvm-commits, >> >> There was a discussion on this topic a while ago, and now I've decided to make a formal proposal and post it here. > > I missed the earlier

On Thu, Jul 26, 2012 at 4:24 PM, Jim Grosbach <grosbach at apple.com> wrote: > > On Jul 26, 2012, at 4:07 PM, Chris Lattner <clattner at apple.com> wrote: > > > <dropping llvm-commits> > > > > On Jul 2, 2012, at 9:59 AM, Alexander Kornienko wrote: > > > >> Hi llvmdev, llvm-commits, > >> > >> There was a discussion on

<dropping llvm-commits> On Jul 2, 2012, at 9:59 AM, Alexander Kornienko wrote: > Hi llvmdev, llvm-commits, > > There was a discussion on this topic a while ago, and now I've decided to make a formal proposal and post it here. I missed the earlier discussion, so I'm sorry for chiming in late. > I propose to add the LLVM_FALLTHROUGH macro for specifying intended

If I compile libFLAC with GCC 7.1 I see many warnings like lpc.c: In function 'FLAC__lpc_compute_residual_from_qlp_coefficients': lpc.c:489:18: warning: this statement may fall through [-Wimplicit-fallthrough=] case 32: sum += qlp_coeff[31] * data[i-32]; ~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~ lpc.c:490:5: note: here case 31: sum += qlp_coeff[30] * data[i-31];

Hi. I upgraded my main build host and the clang -Werror builds started failing. This is because clang 10's -Wimplicit-fallthrough doesn't understand /* FALLTHROUGH */ but rather requires __attribute__((fallthrough)): clang -Wall -O2 [...] -Wimplicit-fallthrough [...] -D_XOPEN_SOURCE=600 -D_BSD_SOURCE -D_DEFAULT_SOURCE -DHAVE_CONFIG_H -c /openbsd-compat/base64.c

On Thu, Jul 26, 2012 at 8:53 PM, Cameron McInally <cameron.mcinally at nyu.edu>wrote: > Hey Alex, > > Sorry if this is a silly question... are you asking if anyone "wants the > functionality proposed" or "wants to write the code for the functionality > proposed"? > *-Wimplicit-fallthrough* diagnostic is already implemented, and the patch in this thread

This is Chris' call to make, imo. -j On Aug 22, 2012, at 3:59 PM, Alexander Kornienko <alexfh at google.com> wrote: > Ping. > > On Thu, Aug 9, 2012 at 3:04 PM, Alexander Kornienko <alexfh at google.com> wrote: > Ping. > > > On Fri, Jul 27, 2012 at 3:42 AM, Alexander Kornienko <alexfh at google.com> wrote: > On Fri, Jul 27, 2012 at 2:02 AM, Richard

http://llvm.org/docs/GettingStarted.html currently mentions Python 2.5 as a minimum required version. I'd like to use argparse <https://docs.python.org/dev/library/argparse.html> in a script and be able to test this script. This requires Python 2.7. This version has been around since 2010, and afaiu, is available on all modern platforms. Is there any reason not to change minimum required

Thank you! It seems like new changes in .rst files are picked up quickly now. -- Alex On Thu, Feb 14, 2019 at 7:33 PM Tanya Lattner <tonic at nondot.org> wrote: > This should all be fixed now. > > Thanks, > Tanya > > > On Feb 14, 2019, at 9:50 AM, Tanya Lattner <tonic at nondot.org> wrote: > > Yes, I am working on it. The sub-projects are still having

Yes, I am working on it. The sub-projects are still having issues but the other documentation has been fixed. -Tanya > On Feb 14, 2019, at 3:53 AM, Alexander Kornienko <alexfh at google.com> wrote: > > Tanya, friendly ping. > > On Mon, Feb 11, 2019 at 3:21 PM Alexander Kornienko <alexfh at google.com <mailto:alexfh at google.com>> wrote: > Hi Tanya, >

While you're here, Xenial (15.10) seems wrong (should be 16.04 LTS). On Wed, Apr 27, 2016 at 12:32 PM, Sylvestre Ledru <sylvestre at debian.org> wrote: > Hello > > Indeed, this package no longer exists. I removed it from llvm.org/apt/ > > Thanks, > Sylvestre > > Le 27/04/2016 à 12:23, Alexander Kornienko a écrit : > > Adding Sylvestre, who should know how

Currently, "clang -help" gives a strange output. This patch fixes it. -------------- next part -------------- A non-text attachment was scrubbed... Name: fix-clang-warning.patch Type: text/x-diff Size: 610 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20080530/1d2c5719/attachment.patch>

Thanks, I fixed the typo with Xenial. I don't know if I want to bring back llvm-jenkins.d.n again yet. if people see an interest in it, why not :) Le 27/04/2016 à 13:06, Alexander Kornienko a écrit : > ... and http://llvm-jenkins.debian.net/ seems to be down. Sorry for > bugging you with all of this, and thanks for fixing the > clang-modernize thingy so quickly! > > On Wed,

On Wed, 2018-02-07 at 23:30 +0000, Chandler Carruth wrote: > This should go to llvm-commits as a proper review. Do you want to do > that David? Want someone on our end to pick it up? I'll attempt to add some test cases... -------------- next part -------------- A non-text attachment was scrubbed... Name: smime.p7s Type: application/x-pkcs7-signature Size: 5213 bytes Desc: not available

Adding Sylvestre, who should know how to fix that. On Tue, Apr 26, 2016 at 10:39 PM, Nick Wiggill <nick.wiggill at gmail.com> wrote: > Thanks for your replies, gentlemen. I'm in good standing with my install, > then. > > On the apt page <http://llvm.org/apt/>, it should not be suggested then > to "apt-get install [..etc..] clang-modernize-3.8 [..etc..]"