[Bug other/105647] New: New test g++.dg/modules/pr105169 from r13-566-g7a3f38a966a528 fails with assembler error

[Bug other/105647] New: New test g++.dg/modules/pr105169 from r13-566-g7a3f38a966a528 fails with assembler error

[seurer at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105647

            Bug ID: 105647
           Summary: New test g++.dg/modules/pr105169 from
                    r13-566-g7a3f38a966a528 fails with assembler error
           Product: gcc
           Version: 13.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: other
          Assignee: unassigned at gcc dot gnu.org
          Reporter: seurer at gcc dot gnu.org
  Target Milestone: ---

g:7a3f38a966a52893fb5bae301a1a3d56961358fb, r13-566-g7a3f38a966a528

FAIL: g++.dg/modules/pr105169 link


commit 7a3f38a966a52893fb5bae301a1a3d56961358fb (HEAD)
Author: Giuliano Belinassi <gbelinassi@suse.de>
Date:   Fri May 6 23:37:52 2022 -0300

    PR105169 Fix references to discarded sections


spawn -ignore SIGHUP
/home/seurer/gcc/git/build/gcc-test/gcc/testsuite/g++9/../../xg++
-B/home/seurer/gcc/git/build/gcc-test/gcc/testsuite/g++9/../../ pr105169_a.s
pr105169_b.s -fdiagnostics-plain-output -nostdinc++
-I/home/seurer/gcc/git/build/gcc-test/powerpc64le-unknown-linux-gnu/libstdc++-v3/include/powerpc64le-unknown-linux-gnu
-I/home/seurer/gcc/git/build/gcc-test/powerpc64le-unknown-linux-gnu/libstdc++-v3/include
-I/home/seurer/gcc/git/gcc-test/libstdc++-v3/libsupc++
-I/home/seurer/gcc/git/gcc-test/libstdc++-v3/include/backward
-I/home/seurer/gcc/git/gcc-test/libstdc++-v3/testsuite/util -fmessage-length=0
-L/home/seurer/gcc/git/build/gcc-test/powerpc64le-unknown-linux-gnu/./libstdc++-v3/src/.libs
-B/home/seurer/gcc/git/build/gcc-test/powerpc64le-unknown-linux-gnu/./libstdc++-v3/src/.libs
-L/home/seurer/gcc/git/build/gcc-test/powerpc64le-unknown-linux-gnu/./libstdc++-v3/src/.libs
-B/home/seurer/gcc/git/build/gcc-test/powerpc64le-unknown-linux-gnu/./libitm/
-L/home/seurer/gcc/git/build/gcc-test/powerpc64le-unknown-linux-gnu/./libitm/.libs
-lm -o ./pr105169.exe^M
pr105169_a.s: Assembler messages:^M
pr105169_a.s:64: Error: .localentry expression for `_Z8functionv' is not a
valid power of 2^M
pr105169_b.s: Assembler messages:^M
pr105169_b.s:67: Error: .localentry expression for
`_ZN21WinsockInterfaceClassC2Ev' is not a valid power of 2^M
compiler exited with status 1
FAIL: g++.dg/modules/pr105169 link

[Bug target/105647] New test g++.dg/modules/pr105169 from r13-566-g7a3f38a966a528 fails with assembler error

[giuliano.belinassi at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105647

--- Comment #1 from Giuliano Belinassi <giuliano.belinassi at gmail dot com> ---
Confirmed.

powerpc64le-suse-linux-gcc-11 -fpatchable-function-entry=1 -O2 -c -o /tmp/a.o
pr105169_a.C

/tmp/ccjfumfv.s: Assembler messages:
/tmp/ccjfumfv.s:64: Error: .localentry expression for `_Z8functionv' is not a
valid power of 2

However, the problem was not introduced in the commit mentioned, as it is
already there since at least gcc-11. Just the testcase that triggers it was
introduced there.

Furthermore, 

powerpc64le-suse-linux-gcc-11 -fpatchable-function-entry=2 -O2 -c -o /tmp/a.o
pr105169_a.C

Works.

[Bug target/105647] New test g++.dg/modules/pr105169 from r13-566-g7a3f38a966a528 fails with assembler error

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105647

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Last reconfirmed|                            |2022-05-19
           Keywords|                            |testsuite-fail
             Status|UNCONFIRMED                 |NEW
     Ever confirmed|0                           |1

--- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
So in fact this is a latent wrong-code by the target support of pfe.

[Bug target/105647] New test g++.dg/modules/pr105169 from r13-566-g7a3f38a966a528 fails with assembler error

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105647

--- Comment #3 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The trunk branch has been updated by Giuliano Belinassi
<giulianob@gcc.gnu.org>:

https://gcc.gnu.org/g:16e02662c092c6e3087628ef0030e7daa2f71485

commit r13-633-g16e02662c092c6e3087628ef0030e7daa2f71485
Author: Giuliano Belinassi <gbelinassi@suse.de>
Date:   Wed May 18 16:16:48 2022 -0300

    PR105647 Update pr105169* so it does not fail on powerpc64le

    On powerpc64le, the tests related to pr105169 failed because the
    .localentry was not on a power of two address due to the extra nop
    instruction taking one byte and thus moving its position one byte
    further. Generating two nops instead moves .localentry to a valid
    position.

    gcc/testsuite/ChangeLog
    2022-05-18  Giuliano Belinassi  <gbelinassi@suse.de>

            PR target/105647
            * g++.dg/modules/pr105169_a.C: Change -fpatchable-function-entry to
2.
            * g++.dg/modules/pr105169_b.C: Likewise.