https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86519
Rainer Orth <ro at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |ro at gcc dot gnu.org
--- Comment #18 from Rainer Orth <ro at gcc dot gnu.org> ---
(In reply to qinzhao from comment #17)
> the patch has been committed as:
>
> https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=263563
The commit message lacked the ChangeLog entry fragment and didns't otherwise
contain the reference to PR testsuite/86519, so bugzilla didn't pick it up.
What's worse, the patch caused the testcase to regress on 32-bit sparc:
+FAIL: gcc.dg/strcmpopt_6.c scan-assembler-times memcmp 2
The file has
sethi %hi(memcmp), %g1
jmp %g1 + %lo(memcmp)
sethi %hi(memcmp), %g1
jmp %g1 + %lo(memcmp)
while for sparcv9 there's only:
call memcmp, 0
call memcmp, 0
Seems scan-assembler-times isn't reliable here.
Rainer
