[valgrind] [Bug 416667] gcc10 ppc64le impossible constraint in 'asm' in test_isa
https://bugs.kde.org/show_bug.cgi?id=416667 Carl Love changed: What|Removed |Added Status|RESOLVED|CLOSED --- Comment #11 from Carl Love --- closed issue -- You are receiving this mail because: You are watching all bug changes.
[valgrind] [Bug 416667] gcc10 ppc64le impossible constraint in 'asm' in test_isa
https://bugs.kde.org/show_bug.cgi?id=416667 --- Comment #10 from Mark Wielaard --- (In reply to Carl Love from comment #9) > commit 4f70aa4fb0d4cef97ee34602d72f3980c5a83bf6 > Author: Carl Love > Date: Tue Jan 28 12:17:16 2020 -0600 > > PPC64: Fix assembler constraints. > > The PPC64 assembler constraints "ws", "wa" and "ww" have changed to "wa" > in GCC 10. Update the ppc64 tests test_isa_2_07_part1.c and > test_isa_3_0.c. > > I believe the issue can be closed if you agree. Works for me. Thanks! -- You are receiving this mail because: You are watching all bug changes.
[valgrind] [Bug 416667] gcc10 ppc64le impossible constraint in 'asm' in test_isa
https://bugs.kde.org/show_bug.cgi?id=416667 Carl Love changed: What|Removed |Added Status|CONFIRMED |RESOLVED Resolution|--- |FIXED --- Comment #9 from Carl Love --- Mark: I pulled down and reviewed your version of the patch. I tested it on PPC64 Power 8 and PPC64 Power 9. It passed fine on both systems. Note, the regression suite was compiled with a pre GCC 10 compiler. I manually compiled the two test cases using a version of the GCC 10 compiler with no issues. The patch was committed to Valgrind mainline. commit 4f70aa4fb0d4cef97ee34602d72f3980c5a83bf6 Author: Carl Love Date: Tue Jan 28 12:17:16 2020 -0600 PPC64: Fix assembler constraints. The PPC64 assembler constraints "ws", "wa" and "ww" have changed to "wa" in GCC 10. Update the ppc64 tests test_isa_2_07_part1.c and test_isa_3_0.c. I believe the issue can be closed if you agree. -- You are receiving this mail because: You are watching all bug changes.
[valgrind] [Bug 416667] gcc10 ppc64le impossible constraint in 'asm' in test_isa
https://bugs.kde.org/show_bug.cgi?id=416667 --- Comment #8 from Mark Wielaard --- (In reply to Mark Wielaard from comment #7) > Created attachment 125383 [details] > GCC10 ppc64 asm constrainst update > > The attached patch is what I am testing now and seems to pass the > compiler/build. Which seems to fix the issue. Both test_isa_2_07_part1.c and test_is_3_0.c compile and the tests pass. -- You are receiving this mail because: You are watching all bug changes.
[valgrind] [Bug 416667] gcc10 ppc64le impossible constraint in 'asm' in test_isa
https://bugs.kde.org/show_bug.cgi?id=416667 --- Comment #7 from Mark Wielaard --- Created attachment 125383 --> https://bugs.kde.org/attachment.cgi?id=125383=edit GCC10 ppc64 asm constrainst update The attached patch is what I am testing now and seems to pass the compiler/build. -- You are receiving this mail because: You are watching all bug changes.
[valgrind] [Bug 416667] gcc10 ppc64le impossible constraint in 'asm' in test_isa
https://bugs.kde.org/show_bug.cgi?id=416667
Mark Wielaard changed:
What|Removed |Added
Status|REPORTED|CONFIRMED
Ever confirmed|0 |1
--- Comment #6 from Mark Wielaard ---
With the patch I am still seeing:
test_isa_2_07_part1.c:398:4: error: impossible constraint in 'asm'
398 |__asm__ __volatile__ ("mtvsrd %x0,%1" : "=ws" (vec_out) : "r"
(r14));
|^~~
test_isa_3_0.c: In function 'test_xscvdphp':
test_isa_3_0.c:1831:4: error: impossible constraint in 'asm'
1831 |__asm__ __volatile__ ("xscvdphp %x0, %x1 " : "+wi" (vec_xt) : "wi"
(vec_xb));
|^~~
I replaced the other "ws" constrainst in test_isa_2_07_part1.c with "wa" too.
For "wi" I found gcc:
commit e670418ff18f36f81f3f619c34578da03d102694
Author: Segher Boessenkool
Date: Tue Jun 4 18:27:45 2019 +0200
rs6000: wi->wa, wt->wa
"wi" and "wt" mean just the same as "wa" these days. Change them to
the simpler name.
And
commit 72e3386e136205dcc69e61e538b94643832cf22a
Author: Segher Boessenkool
Date: Wed Jun 5 01:31:32 2019 +0200
rs6000: ww -> wa
"ww" can always be "wa".
So I did that in test_isa_3_0.c
--
You are receiving this mail because:
You are watching all bug changes.
[valgrind] [Bug 416667] gcc10 ppc64le impossible constraint in 'asm' in test_isa
https://bugs.kde.org/show_bug.cgi?id=416667 --- Comment #5 from Carl Love --- Created attachment 125376 --> https://bugs.kde.org/attachment.cgi?id=125376=edit Patch, fix assembler constraints The attached patch changes the ww and ws assembler constraints to wa. Please test this patch in your environment and let me know if it fixes all of the issues. I am only able to partially reproduce the errors you saw in the initial bug report. Thanks. -- You are receiving this mail because: You are watching all bug changes.
[valgrind] [Bug 416667] gcc10 ppc64le impossible constraint in 'asm' in test_isa
https://bugs.kde.org/show_bug.cgi?id=416667 --- Comment #4 from Carl Love --- Julian, Mark: Looks like it is a GCC change. Per the message from Will It's related to some cleanups that Segher did last year. git show cc998fd5f43a296e1a12bf4de63c4c9dd1d39cfa git show 208a040511b9c4d9a59af1caafa855a031a7a0ca git show 0d0863136f59e7f937f60c772921bc73708fba81 "ws" is just "wa". and "wv" is "v", but only if VSX is enabled (otherwise it's NO_REGS). and rs6000: ww->wa in testsuite -- You are receiving this mail because: You are watching all bug changes.
[valgrind] [Bug 416667] gcc10 ppc64le impossible constraint in 'asm' in test_isa
https://bugs.kde.org/show_bug.cgi?id=416667
--- Comment #3 from Carl Love ---
Julian:
I hadn't thought to try different optimizations. I can reproduce the issue for
the mfvsrd test only with no optimizations. No errors are reported with -O1,
-O2 and -O3 optimization levels.
$GCC_INSTALL/bin/gcc -c valgrind-bug.c
valgrind-bug.c: In function ‘test_mfvsrd’:
valgrind-bug.c:19:4: error: impossible constraint in ‘asm’
19 |__asm__ __volatile__ ("mfvsrd %0,%x1" : "=r" (r14) : "ws"
(vec_inA));
|^~~
$GCC_INSTALL/bin/gcc -O0 -c valgrind-bug.c
valgrind-bug.c: In function ‘test_mfvsrd’:
valgrind-bug.c:19:4: error: impossible constraint in ‘asm’
19 |__asm__ __volatile__ ("mfvsrd %0,%x1" : "=r" (r14) : "ws"
(vec_inA));
|^~~
~/Valgrind$ $GCC_INSTALL/bin/gcc -O1 -c valgrind-bug.c
~/Valgrind$ $GCC_INSTALL/bin/gcc -O2 -c valgrind-bug.c
~/Valgrind$ $GCC_INSTALL/bin/gcc -O3 -c valgrind-bug.c
--
You are receiving this mail because:
You are watching all bug changes.
[valgrind] [Bug 416667] gcc10 ppc64le impossible constraint in 'asm' in test_isa
https://bugs.kde.org/show_bug.cgi?id=416667 --- Comment #2 from Julian Seward --- (In reply to Carl Love from comment #1) Carl, > I was able to reproduce the first issue but not > the second. I sent email to the compiler team about the issue. Wondering > if there is an assembler change causing the issue. I wouldn't have thought this was an assembler issue; the complaint occurs before assembly code is ever produced. I wonder if it might be to do with compiler optimisation level though. Can you reproduce it at any of -O0, -Og, -O1, -O2, -O3 ? -- You are receiving this mail because: You are watching all bug changes.
[valgrind] [Bug 416667] gcc10 ppc64le impossible constraint in 'asm' in test_isa
https://bugs.kde.org/show_bug.cgi?id=416667 --- Comment #1 from Carl Love --- Mark: I copied the two tests into a file and tried compiling them on a GCC10 compiler I have available. I was able to reproduce the first issue but not the second. I sent email to the compiler team about the issue. Wondering if there is an assembler change causing the issue. Anyway, just wanted to let you know I saw your bug and am looking into it. -- You are receiving this mail because: You are watching all bug changes.
[valgrind] [Bug 416667] gcc10 ppc64le impossible constraint in 'asm' in test_isa
https://bugs.kde.org/show_bug.cgi?id=416667 Mark Wielaard changed: What|Removed |Added CC||c...@us.ibm.com, ||will_schm...@vnet.ibm.com -- You are receiving this mail because: You are watching all bug changes.
