This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][Middle-end]patch for fixing PR 86519
On 08/17/2018 09:43 PM, Paul Hua wrote:
> Hi Qing:
>
>>
>> the change has been committed as:
>> https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=263563 <https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=263563>
>>
>> Qing
>>
>
> The strcmpopt_6.c test still fails on mips64el target.
>
> gcc.dg/strcmpopt_6.c: memcmp found 4 times
> FAIL: gcc.dg/strcmpopt_6.c scan-assembler-times memcmp 2
>
>
> The mips asm output have ".reloc" info.
>
> -------------------------------------------------------------------------
> ld $5,%got_page(.LC0)($28)
> ld $25,%call16(memcmp)($28)
> li $6,3 # 0x3
> sd $31,8($sp)
> .reloc 1f,R_MIPS_JALR,memcmp
> 1: jalr $25
> daddiu $5,$5,%got_ofst(.LC0)
> ----------------------------------------------------------------------------
>
> scan-assembler find "4" times.
Ugh. :( There's probably other targets that are going to have similar
properties. I'm not really sure how to write a suitable assembler test
for this.
Maybe we just need a different search result for MIPS (and potentially
other targets).
jeff