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

Hans Wennborg via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 27 15:07:23 PST 2017


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.
>> >
>> >
>
>


More information about the llvm-commits mailing list