Re: [PATCH] Add test-case (PR ipa/84805).

2018-03-20 Thread Jeff Law 
On 03/14/2018 11:23 AM, Martin Liška wrote:
> Hi.
> 
> This is a new test-case isolated from libreoffice.
> 
> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
> I'm going to install it.
> 
> Martin
> 
> gcc/testsuite/ChangeLog:
> 
> 2018-03-14  Martin Liska  
> 
>   PR ipa/8480
>   * g++.dg/lto/pr84805_0.C: New test.
>   * g++.dg/lto/pr84805_1.C: New test.
>   * g++.dg/lto/pr84805_2.C: New test.
OK if you test on other targets and they're passing -- per Eric B's warning.

jeff

Re: [PATCH] Add test-case (PR ipa/84805).

2018-03-15 Thread Eric Botcazou 
> Sorry for the breakage. Actually I did testing on x86_64 and ppcl64
> machines.

There is no breakage, just an issue with the ordering of warnings which may 
vary across platforms.

-- 
Eric Botcazou

Re: [PATCH] Add test-case (PR ipa/84805).

2018-03-15 Thread Martin Liška 
On 03/14/2018 09:12 PM, Eric Botcazou wrote:
>> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
> 
> Please make sure to test it on more platforms (see PR ipa/83983 for details).
> 

Sorry for the breakage. Actually I did testing on x86_64 and ppcl64 machines.

Martin

Re: [PATCH] Add test-case (PR ipa/84805).

2018-03-15 Thread Christophe Lyon 
On 14 March 2018 at 21:12, Eric Botcazou  wrote:
>> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
>
> Please make sure to test it on more platforms (see PR ipa/83983 for details).
>

Hi,

These new tests do not work well on arm-none-eabi and aarch64-none-elf.
(as opposed to arm*linux* and aarch64*linux*)

On arm-none-eabi, I see:
/aci-gcc-fsf/builds/gcc-fsf-gccsrc/tools/arm-none-eabi/bin/ld:
/aci-gcc-fsf/builds/gcc-fsf-gccsrc/obj-arm-none-eabi/gcc3/gcc/testsuite/g++1/../../crtbegin.o:
relocation R_ARM_MOVW_ABS_NC against `a local symbol' can not be used
when making a shared object; recompile with -fPIC
/aci-gcc-fsf/builds/gcc-fsf-gccsrc/obj-arm-none-eabi/gcc3/gcc/testsuite/g++1/../../crtbegin.o:
error adding symbols: Bad value
collect2: error: ld returned 1 exit status
compiler exited with status 1
FAIL: g++.dg/lto/pr84805  (test for LTO warnings, pr84805_0.C line 106)
FAIL: g++.dg/lto/pr84805  (test for LTO warnings, pr84805_0.C line 142)
FAIL: g++.dg/lto/pr84805  (test for LTO warnings, pr84805_0.C line 147)
FAIL: g++.dg/lto/pr84805  (test for LTO warnings, pr84805_1.C line 6)
FAIL: g++.dg/lto/pr84805  (test for LTO warnings, pr84805_2.C line 18)
FAIL: g++.dg/lto/pr84805 cp_lto_pr84805_0.o-cp_lto_pr84805_2.o link,
-O2 -fPIC -shared -flto

On aarch64-none-elf, I see:
/aci-gcc-fsf/builds/gcc-fsf-gccsrc/tools/aarch64-none-elf/bin/ld:
/aci-gcc-fsf/builds/gcc-fsf-gccsrc/tools/lib/gcc/aarch64-none-elf/8.0.1/../../../../aarch64-none-elf/lib/librdimon.a(rdimon-syscalls.o):
relocation R_AARCH64_ADR_PREL_PG_HI21 against external symbol
`_impure_ptr' can not be used when making a shared object; recompile
with -fPIC
PASS: g++.dg/lto/pr84805  (test for LTO warnings, pr84805_0.C line 106)
PASS: g++.dg/lto/pr84805  (test for LTO warnings, pr84805_0.C line 142)
PASS: g++.dg/lto/pr84805  (test for LTO warnings, pr84805_0.C line 147)
PASS: g++.dg/lto/pr84805  (test for LTO warnings, pr84805_1.C line 6)
PASS: g++.dg/lto/pr84805  (test for LTO warnings, pr84805_2.C line 18)
FAIL: g++.dg/lto/pr84805 cp_lto_pr84805_0.o-cp_lto_pr84805_2.o link,
-O2 -fPIC -shared -flto

Christophe


> --
> Eric Botcazou

Re: [PATCH] Add test-case (PR ipa/84805).

2018-03-14 Thread Eric Botcazou 
> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.

Please make sure to test it on more platforms (see PR ipa/83983 for details).

-- 
Eric Botcazou