r345660 - [clang][ubsan] Implicit Conversion Sanitizer - integer sign change - clang part

[Roman Lebedev via cfe-commits]

On Thu, Nov 1, 2018 at 9:39 AM Roman Lebedev <lebedev.ri at gmail.com> wrote:
>
> On Thu, Nov 1, 2018 at 3:03 AM Friedman, Eli <efriedma at codeaurora.org> wrote:
> >
> > On 10/30/2018 2:58 PM, Roman Lebedev via cfe-commits wrote:
> > > Author: lebedevri
> > > Date: Tue Oct 30 14:58:56 2018
> > > New Revision: 345660
> > >
> > > URL: http://llvm.org/viewvc/llvm-project?rev=345660&view=rev
> > > Log:
> > > [clang][ubsan] Implicit Conversion Sanitizer - integer sign change - clang part
> > >
> > > This is the second half of Implicit Integer Conversion Sanitizer.
> > > It completes the first half, and finally makes the sanitizer
> > > fully functional! Only the bitfield handling is missing.
> >
> > This is causing failures on the polly-aosp buildbot
> > (http://lab.llvm.org:8011/builders/aosp-O3-polly-before-vectorizer-unprofitable/builds/690/steps/build-aosp/logs/stdio
> > ).  I haven't tried to reduce the issue yet; let me know if you need help.
> Thanks!
> I'm able to cause a similar crash, will handle it from here.
> It *should* be trivial/fast to triage, so maybe the revert can be
> avoided. Or not.
Fixed in r345816.

> > -Eli
Roman.

> > --
> > Employee of Qualcomm Innovation Center, Inc.
> > Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
> >