On 05/21/2018 04:42 PM, Sudakshina Das wrote:
> On 21/05/18 15:00, Rainer Orth wrote:
>> Hi Martin,
>>
>>>>> Thanks for opened eyes, following patch will fix that.
>>>>> It's quite obvious, I'll install it right after tests will finish.
>>>>
>>>> unfortunately, it didn't fix either issue:
>>>>
>>>> * The switchlower -> switchlower1 renames in the dg-final* lines
>>>> (attached) are still necessary to avoid the UNRESOLVED errors.
>>>> Although obvious, I haven't installed them since ...
>>>>
>>>> * ... even so
>>>>
>>>> FAIL: gcc.dg/tree-prof/update-loopch.c scan-tree-dump switchlower1
>>>> "Removing basic block"
>>>>
>>>> remains.
>> [...]
>>> You are right, it's using -O2, thus your patch is right. Please install the
>>> patch
>>> after testing. It's obvious fix.
>>
>> But what about the remaining FAIL?
>>
>
> Sorry to add to this, but I have also observed the following failures on
> aarch64-none-elf, aarch64-none-linux-gnu and aarch64_be-none-elf targets
> bisected to this commit:
>
> FAIL: gcc.dg/sancov/cmp0.c -O0 scan-tree-dump-times optimized
> "__builtin___sanitizer_cov_trace_const_cmp" 7
>
> FAIL: gcc.dg/sancov/cmp0.c -O0 scan-tree-dump-times optimized
> "__builtin___sanitizer_cov_trace_switch \\(" 2
>
> FAIL: gcc.dg/sancov/cmp0.c -O0 -g scan-tree-dump-times optimized
> "__builtin___sanitizer_cov_trace_const_cmp" 7
>
> FAIL: gcc.dg/sancov/cmp0.c -O0 -g scan-tree-dump-times optimized
> "__builtin___sanitizer_cov_trace_switch \\(" 2
Hi.
I've just tested sancov tests on my aarch64 and cmp0.c looks fine. Can you
please tell me which -march, -mtune does
your board have?
>
> FAIL: gcc.dg/tree-ssa/pr77445-2.c scan-tree-dump-not thread3 "not considered"
>
> FAIL: gcc.dg/tree-ssa/ssa-dom-thread-7.c scan-tree-dump-not vrp2 "Jumps
> threaded"
I can confirm these 2. It's kind of expected, I will clean it up before next
release. Jeff is aware
of that..
Martin
>
> Sudi
>
>> Rainer
>>
>
