On 03/01/2018 04:53 AM, Martin Liška wrote:
> On 03/01/2018 12:45 PM, Jakub Jelinek wrote:
>> On Thu, Mar 01, 2018 at 12:41:37PM +0100, Martin Liška wrote:
>>> I've been running periodically UBSAN bootstrap and as the runtime errors are
>>> not causing failure of compiler I haven't noticed the errors.
>>> Thus I would like to disable UBSAN recovery. Apart from that I'm handling
>>> issue in md5.c where in UBSAN bootstrap we want to do proper pointer 
>>> alignment.
>>> Doing that 2 will be remaining issues that will block the bootstrap:
>>>
>>> PR84634 and PR84635
>>>
>>> May I then install the patch? Jakub what do you think about it?
>> I actually prefer recovery where possible, that way we can get more runtime
>> errors at once, rather than stopping at the first one.
>> You always can and should grep the build logs.
> Ok, you convinced me with Marek :)
> 
>> As for md5.c, you only posted ChangeLog for it, not the actual patch.
> Sorry, I'm attaching updated patch.
> 
> Martin
> 
>>      Jakub
>>
> 
> 0001-Improve-boostrap-ubsan-config-PR-bootstrap-64914.patch
> 
> 
> From ccf9285022df7a2c11c14e5dc388d668d18243eb Mon Sep 17 00:00:00 2001
> From: marxin <mli...@suse.cz>
> Date: Thu, 1 Mar 2018 12:13:34 +0100
> Subject: [PATCH] Improve boostrap-ubsan config (PR bootstrap/64914).
> 
> config/ChangeLog:
> 
> 2018-03-01  Martin Liska  <mli...@suse.cz>
> 
>       PR bootstrap/64914
>       * bootstrap-ubsan.mk: Define UBSAN_BOOTSTRAP.
> 
> libiberty/ChangeLog:
> 
> 2018-03-01  Martin Liska  <mli...@suse.cz>
> 
>       PR bootstrap/64914
>       * md5.c: Use strict alignment with UBSAN_BOOTSTRAP.
OK
jeff

Reply via email to