[PATCH] D39085: [PM] Wire up support for the bounds checking sanitizer with the new PM.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 13 21:20:32 PST 2017


Should be fixed in r318137, but will have to wait for the bot to catch up
to tell for sure.

On Mon, Nov 13, 2017 at 9:17 PM Maxim Kuvyrkov <maxim.kuvyrkov at linaro.org>
wrote:

> Thanks, Chandler.
> --
> Maxim Kuvyrkov
> www.linaro.org
>
>
>
> > On Nov 14, 2017, at 8:07 AM, Chandler Carruth <chandlerc at gmail.com>
> wrote:
> >
> > Sorry, there is another unrelated error I've found, and am fixing....
> Appears to be ARM build bot specific.
> >
> > On Mon, Nov 13, 2017 at 9:05 PM Chandler Carruth <chandlerc at gmail.com>
> wrote:
> > It has already been fixed in r318130.
> >
> > On Mon, Nov 13, 2017 at 9:02 PM Maxim Kuvyrkov via llvm-commits <
> llvm-commits at lists.llvm.org> wrote:
> > > On Nov 14, 2017, at 5:00 AM, Chandler Carruth via Phabricator via
> llvm-commits <llvm-commits at lists.llvm.org> wrote:
> > >
> > > This revision was automatically updated to reflect the committed
> changes.
> > > Closed by commit rL318131: [PM] Wire up support for the bounds
> checking sanitizer with the new PM. (authored by chandlerc).
> > >
> > > Changed prior to commit:
> > >  https://reviews.llvm.org/D39085?vs=119558&id=122760#toc
> > >
> > > Repository:
> > >  rL LLVM
> > >
> > > https://reviews.llvm.org/D39085
> > >
> > > Files:
> > >  cfe/trunk/lib/CodeGen/BackendUtil.cpp
> > >  cfe/trunk/test/CodeGen/bounds-checking.c
> > >
> > >
> > ...
> > > Index: cfe/trunk/test/CodeGen/bounds-checking.c
> > > ===================================================================
> > > --- cfe/trunk/test/CodeGen/bounds-checking.c
> > > +++ cfe/trunk/test/CodeGen/bounds-checking.c
> > > @@ -1,5 +1,7 @@
> > > // RUN: %clang_cc1 -fsanitize=local-bounds -emit-llvm -triple
> x86_64-apple-darwin10 %s -o - | FileCheck %s
> > > +// RUN: %clang_cc1 -fsanitize=local-bounds
> -fexperimental-new-pass-manager -emit-llvm -triple x86_64-apple-darwin10 %s
> -o - | FileCheck %s
> > > // RUN: %clang_cc1 -fsanitize=array-bounds -O
> -fsanitize-trap=array-bounds -emit-llvm -triple x86_64-apple-darwin10
> -DNO_DYNAMIC %s -o - | FileCheck %s
> > > +// RUN: %clang_cc1 -fsanitize=array-bounds -O
> -fsanitize-trap=array-bounds -fexperimental-new-pass-manager -emit-llvm
> -triple x86_64-apple-darwin10 -DNO_DYNAMIC %s -o - | FileCheck %s
> >
> > Hi Chandler,
> >
> > This seems to break armv7 buildbots (
> http://lab.llvm.org:8011/builders/clang-cmake-thumbv7-a15/builds/13050).
> Would you please investigate?
> >
> > --
> > Maxim Kuvyrkov
> > www.linaro.org
> >
> >
> >
> >
> > >
> > > // CHECK-LABEL: @f
> > > double f(int b, int i) {
> > >
> > >
> > > <D39085.122760.patch>_______________________________________________
> > > llvm-commits mailing list
> > > llvm-commits at lists.llvm.org
> > > http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at lists.llvm.org
> > http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171114/a27aa418/attachment.html>


More information about the llvm-commits mailing list