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

2018-11-01 Thread Roman Lebedev via cfe-commits
On Thu, Nov 1, 2018 at 9:39 AM Roman Lebedev wrote: > > On Thu, Nov 1, 2018 at 3:03 AM Friedman, Eli 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/

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

2018-10-31 Thread Roman Lebedev via cfe-commits
On Thu, Nov 1, 2018 at 3:03 AM Friedman, Eli 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] Imp

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

2018-10-31 Thread Friedman, Eli via cfe-commits
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

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

2018-10-30 Thread Roman Lebedev via cfe-commits
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 th