From: Giuliano Belinassi Date: Wed, 18 May 2022 19:16:48 +0000 (-0300) Subject: PR105647 Update pr105169* so it does not fail on powerpc64le X-Git-Tag: basepoints/gcc-14~6574 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=16e02662c092c6e3087628ef0030e7daa2f71485;p=thirdparty%2Fgcc.git 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 PR target/105647 * g++.dg/modules/pr105169_a.C: Change -fpatchable-function-entry to 2. * g++.dg/modules/pr105169_b.C: Likewise. --- diff --git a/gcc/testsuite/g++.dg/modules/pr105169_a.C b/gcc/testsuite/g++.dg/modules/pr105169_a.C index 66dc4b7901f..02660b3a0e4 100644 --- a/gcc/testsuite/g++.dg/modules/pr105169_a.C +++ b/gcc/testsuite/g++.dg/modules/pr105169_a.C @@ -1,6 +1,6 @@ /* { dg-module-do link } */ -/* { dg-options "-std=c++11 -fpatchable-function-entry=1 -O2" } */ -/* { dg-additional-options "-std=c++11 -fpatchable-function-entry=1 -O2" } */ +/* { dg-options "-std=c++11 -fpatchable-function-entry=2 -O2" } */ +/* { dg-additional-options "-std=c++11 -fpatchable-function-entry=2 -O2" } */ /* This test is in the "modules" package because it supports multiple files linkage. */ diff --git a/gcc/testsuite/g++.dg/modules/pr105169_b.C b/gcc/testsuite/g++.dg/modules/pr105169_b.C index 5f8b00dfe51..7a9c5863a6a 100644 --- a/gcc/testsuite/g++.dg/modules/pr105169_b.C +++ b/gcc/testsuite/g++.dg/modules/pr105169_b.C @@ -1,6 +1,6 @@ /* { dg-module-do link } */ -/* { dg-options "-std=c++11 -fpatchable-function-entry=1 -O2" } */ -/* { dg-additional-options "-std=c++11 -fpatchable-function-entry=1 -O2" } */ +/* { dg-options "-std=c++11 -fpatchable-function-entry=2 -O2" } */ +/* { dg-additional-options "-std=c++11 -fpatchable-function-entry=2 -O2" } */ /* This test is in the "modules" package because it supports multiple files linkage. */