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