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

[Maxim Kuvyrkov via llvm-commits]

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