On Wed, 24 Oct 2018, Rainer Orth wrote:
> Hi Eric,
>
> >> I know: a patch to fix this is almost ready, just needs a final round of
> >> testing.
> >
> > OK, thanks for the information.
>
> here's what I've got. It took me two false starts, unfortunately:
>
> * Initially, I just tried linking w
Hi Eric,
>> I know: a patch to fix this is almost ready, just needs a final round of
>> testing.
>
> OK, thanks for the information.
here's what I've got. It took me two false starts, unfortunately:
* Initially, I just tried linking with LD_OPTIONS='-z nocompstrtab':
-z nocompstrtab
> I know: a patch to fix this is almost ready, just needs a final round of
> testing.
OK, thanks for the information.
--
Eric Botcazou
Hi Eric,
>> it's not disabled (I had to disable it when testing an a /bin/as version
>> with full SHF_MERGE/SHF_STRINGS suppurt recently), so I suspect the
>> latter. In S11.4 .rodata and .rodata.str1.8 are merged, with the
>> alignment of the larger of the two on the output section.
>
> OK. The
> it's not disabled (I had to disable it when testing an a /bin/as version
> with full SHF_MERGE/SHF_STRINGS suppurt recently), so I suspect the
> latter. In S11.4 .rodata and .rodata.str1.8 are merged, with the
> alignment of the larger of the two on the output section.
OK. The regressions are
Hi Rainer,
On 10/9/18 3:05 PM, Rainer Orth wrote:
> Hi Bernd,
>
>>> * The merge-all-constants-2.c test doesn't FAIL on Solaris/SPARC with
>>> /bin/as, although it lacks string merging support, too. The assembler
>>> output contains
>>>
>>> .section".rodata"
>>>
>>>
On 10/9/18 6:45 AM, Bernd Edlinger wrote:
> On 10/03/18 18:31, Jeff Law wrote:
>>> - && (len = int_size_in_bytes (TREE_TYPE (decl))) > 0
>>> - && TREE_STRING_LENGTH (decl) >= len)
>>> + && (len = int_size_in_bytes (TREE_TYPE (decl))) >= 0
>>> + && TREE_STRING_LENGTH (decl) == le
Hi Eric,
>> This isn't necessary on Solaris 11.4, and Solaris 11.3/x86 isn't
>> affected as well. I'm still determining what the best course of action
>> is: disable string merging support before Solaris 11.4 or enable the
>> workaround above instead.
>
> Out of curiosity, why isn't it necessary
> Looking at the .rodata.str1.4 section, I see
>
> $ objdump -s -j .rodata.str1.8 allocatable_function_5.exe
>
> allocatable_function_5.exe: file format elf32-sparc-sol2
>
> Contents of section .rodata.str1.8:
> 10ba8 6d666f6f 2063616c 6c696e67 2000 mfoo calling ...
> 10bb8 666f6f00 0
Hi Eric,
>> Which version exactly (pkg list entire) of Solaris 11 are you running?
>> I'm using gas 2.31 and /bin/ld on Solaris 11.4 resp. 11.5 Beta, where
>> Bernd's patch in PR bootstrap/87551 fixed the remaining regressions.
>
> Solaris 11.3 with Gas 2.30.
I could now reproduce the regressions
> Which version exactly (pkg list entire) of Solaris 11 are you running?
> I'm using gas 2.31 and /bin/ld on Solaris 11.4 resp. 11.5 Beta, where
> Bernd's patch in PR bootstrap/87551 fixed the remaining regressions.
Solaris 11.3 with Gas 2.30.
--
Eric Botcazou
Hi Bernd,
>> * The merge-all-constants-2.c test doesn't FAIL on Solaris/SPARC with
>>/bin/as, although it lacks string merging support, too. The assembler
>>output contains
>>
>> .section".rodata"
>>
>>so the pattern currently used to check for .rodata is too
>>
On 10/03/18 18:31, Jeff Law wrote:
>> - && (len = int_size_in_bytes (TREE_TYPE (decl))) > 0
>> - && TREE_STRING_LENGTH (decl) >= len)
>> + && (len = int_size_in_bytes (TREE_TYPE (decl))) >= 0
>> + && TREE_STRING_LENGTH (decl) == len)
> Not sure why you want to test for >= 0 here
Hi Eric,
>> Besides, the patch seems to have produced more fallout on Solaris: I see
>> many new Go testsuite failures on Solaris 10 which probably are related,
>> and Solaris bootstrap with Ada included is broken due to the extended
>> usage of string merging. I'm currently investigating what's
> Besides, the patch seems to have produced more fallout on Solaris: I see
> many new Go testsuite failures on Solaris 10 which probably are related,
> and Solaris bootstrap with Ada included is broken due to the extended
> usage of string merging. I'm currently investigating what's going on
> the
On 10/08/18 13:14, Rainer Orth wrote:
> Hi Bernd,
>
>> On 10/05/18 20:15, Andreas Schwab wrote:
>>> On Sep 14 2018, Bernd Edlinger wrote:
>>>
diff -Npur gcc/testsuite/gnat.dg/string_merge1.adb
gcc/testsuite/gnat.dg/string_merge1.adb
--- gcc/testsuite/gnat.dg/string_merge1.adb 1970-
Hi Bernd,
> Besides, the patch seems to have produced more fallout on Solaris: I see
> many new Go testsuite failures on Solaris 10 which probably are related,
> and Solaris bootstrap with Ada included is broken due to the extended
> usage of string merging. I'm currently investigating what's goi
Hi Bernd,
> On 10/05/18 20:15, Andreas Schwab wrote:
>> On Sep 14 2018, Bernd Edlinger wrote:
>>
>>> diff -Npur gcc/testsuite/gnat.dg/string_merge1.adb
>>> gcc/testsuite/gnat.dg/string_merge1.adb
>>> --- gcc/testsuite/gnat.dg/string_merge1.adb 1970-01-01
>>> 01:00:00.0 +0100
>>> +++ gcc/
On 10/05/18 20:15, Andreas Schwab wrote:
> On Sep 14 2018, Bernd Edlinger wrote:
>
>> diff -Npur gcc/testsuite/gnat.dg/string_merge1.adb
>> gcc/testsuite/gnat.dg/string_merge1.adb
>> --- gcc/testsuite/gnat.dg/string_merge1.adb 1970-01-01 01:00:00.0
>> +0100
>> +++ gcc/testsuite/gnat.dg
On Sep 14 2018, Bernd Edlinger wrote:
> diff -Npur gcc/testsuite/gnat.dg/string_merge1.adb
> gcc/testsuite/gnat.dg/string_merge1.adb
> --- gcc/testsuite/gnat.dg/string_merge1.adb 1970-01-01 01:00:00.0
> +0100
> +++ gcc/testsuite/gnat.dg/string_merge1.adb 2018-08-26 16:31:12.65027193
On 9/14/18 12:39 PM, Bernd Edlinger wrote:
> Hi,
>
> this is an upate of the string-merge section, it is based on the V2-STRING_CST
> semantic patch series, which was finally installed yesterday.
> It merges single-byte string constants with or without terminating NUL.
> The patch has the same Ada
On Fri, Sep 21, 2018 at 11:36:53AM +, Bernd Edlinger wrote:
> are your concerns addressed with this answer, or do you have objections
> to this patch?
No objections, but no time to review your patch either right now.
Jakub
Hi Jakub,
are your concerns addressed with this answer, or do you have objections
to this patch?
Thanks
Bernd.
On 09/14/18 21:06, Bernd Edlinger wrote:
> On 09/14/18 21:01, Jakub Jelinek wrote:
>> On Fri, Sep 14, 2018 at 06:39:38PM +, Bernd Edlinger wrote:
>>> Hi,
>>>
>>> this is an upate
On 09/14/18 21:01, Jakub Jelinek wrote:
> On Fri, Sep 14, 2018 at 06:39:38PM +, Bernd Edlinger wrote:
>> Hi,
>>
>> this is an upate of the string-merge section, it is based on the
>> V2-STRING_CST
>> semantic patch series, which was finally installed yesterday.
>> It merges single-byte string
On Fri, Sep 14, 2018 at 06:39:38PM +, Bernd Edlinger wrote:
> Hi,
>
> this is an upate of the string-merge section, it is based on the V2-STRING_CST
> semantic patch series, which was finally installed yesterday.
> It merges single-byte string constants with or without terminating NUL.
> The p
25 matches
Mail list logo
