[llvm] r291918 - Fix UBSan bots by blacklisting bits/stl_tree.h.

Ivan Krasin via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 27 15:08:12 PST 2017


thanks!

On Fri, Jan 27, 2017 at 3:07 PM, Hans Wennborg <hans at chromium.org> wrote:

> Merged in r293342.
>
> On Fri, Jan 27, 2017 at 2:41 PM, Ivan Krasin <krasin at google.com> wrote:
> > Then it makes sense, but there is no strong sense of urgency here.
> >
> > On Fri, Jan 27, 2017 at 2:38 PM, Hans Wennborg <hans at chromium.org>
> wrote:
> >>
> >> The source is also released, so a user might build it with UBSan
> >> themselves, perhaps as part of a larger project they'd like to be
> >> UBSan clean?
> >>
> >> On Fri, Jan 27, 2017 at 9:58 AM, Ivan Krasin <krasin at google.com> wrote:
> >> > Probably not. This blacklist only applies to Clang profiled with
> UBSan,
> >> > which is not what we release. The end users won't see or use this
> >> > blacklist.
> >> > Sorry, if the naming is confusing.
> >> >
> >> > On Fri, Jan 27, 2017 at 9:54 AM, Hans Wennborg <hans at chromium.org>
> >> > wrote:
> >> >>
> >> >> On Fri, Jan 13, 2017 at 9:30 AM, Ivan Krasin via llvm-commits
> >> >> <llvm-commits at lists.llvm.org> wrote:
> >> >> > Author: krasin
> >> >> > Date: Fri Jan 13 11:30:10 2017
> >> >> > New Revision: 291918
> >> >> >
> >> >> > URL: http://llvm.org/viewvc/llvm-project?rev=291918&view=rev
> >> >> > Log:
> >> >> > Fix UBSan bots by blacklisting bits/stl_tree.h.
> >> >> >
> >> >> > Summary:
> >> >> > libstdc++ has some undefined behavior in bits/stl_tree.h that
> >> >> > has recently became excercised by some of the LLVM code.
> >> >> > Given that fixing libstdc++ will take years, adding the file
> >> >> > into a blacklist to fix bots seems like a necessity.
> >> >> >
> >> >> > Reviewers: vitalybuka
> >> >> >
> >> >> > Subscribers: llvm-commits, mgorny
> >> >> >
> >> >> > Differential Revision: https://reviews.llvm.org/D28686
> >> >>
> >> >> Should we merge this to 4.0?
> >> >>
> >> >> Probably not a huge benefit, but it seems pretty risk free, and maybe
> >> >> nice to have if someone tries to run ubsan on the released version.
> >> >
> >> >
> >
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170127/1e5781dd/attachment.html>


More information about the llvm-commits mailing list