https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97543
--- Comment #5 from Michael Meissner ---
One of my patches for adding IEEE 128-bit long double may help with this
situation. The ibm-ldouble.c module was not being compiled with
-mno-gnu-attributes would affect things if a different long double
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97543
--- Comment #7 from Michael Meissner ---
Just to be clear, my patch only turns on -mno-gnu-attributes on compiling
ibm-ldouble.c. That is the module that has the extended IBM 128-bit support in
it.
However, I believe if any module in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97543
--- Comment #8 from Michael Meissner ---
In addition to ibm-ldouble.c, the following functions set the gnu attribute #4
to 5 (i.e. pass/use IBM extended double as long double):
_divtc3
_fixtfdi
_fixunstfdi
_floatditf
_floatunditf
_multc3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96791
--- Comment #22 from Michael Meissner ---
When I wrote the original in power7 days, the intent was:
If the user said -mcpu=power7 (for 32-bit) and did not explicitly set either
-mabi=altivec or -mabi=no-altivec, that -mabi=altivec would be set
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97791
Bug ID: 97791
Summary: GNU attributes for long double could be improved on
PowerPC
Product: gcc
Version: 11.0
Status: UNCONFIRMED
Severity: normal
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97653
Michael Meissner changed:
What|Removed |Added
Last reconfirmed||2020-11-03
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97653
--- Comment #4 from Michael Meissner ---
You need the patch that fixes PR libgcc/97543, which is another side of the
same coin. PR 97543 was about making long double default to 64-bits, PR 97643
is about making long double default to IEEE
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98519
--- Comment #9 from Michael Meissner ---
I agree with Segher. Given the 'magic' we need to add the missing 'p' and set
the length for normal RTL, I don't see any reasonable way to add it to asm. We
will just need to use a hook (or make one) to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98519
--- Comment #15 from Michael Meissner ---
FWIW, the hook that will need to be modified is rs6000_md_asm_adjust in
rs6000.c. It appears you are passed the outputs, the inputs, the constraints,
and the clobbers. Right now, we just mark the CA
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98519
--- Comment #2 from Michael Meissner ---
That fell off the plate. I imagine we are going to need a hook with asm that
makes sure none of the memory references are PC-relative.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97653
Michael Meissner changed:
What|Removed |Added
Resolution|--- |FIXED
Status|ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97543
Michael Meissner changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97653
Bug 97653 depends on bug 97543, which changed state.
Bug 97543 Summary: powerpc64le: libgcc has unexpected long double in
.gnu_attribute
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97543
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98645
Michael Meissner changed:
What|Removed |Added
CC||bergner at gcc dot gnu.org,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98645
Bug ID: 98645
Summary: C++ modules support does not work on PowerPC with IEEE
128-bit long double
Product: gcc
Version: 11.0
Status: UNCONFIRMED
Severity:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98874
Bug ID: 98874
Summary: PowerPC test
gcc.target/powerpc/ppc-fortran/pr80108-1.f90 uses
wrong dg-options
Product: gcc
Version: 11.0
Status: UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98873
Bug ID: 98873
Summary: PowerPC test
gcc.target/powerpc/ppc-fortran/ieee128-math.f90 now
fails
Product: gcc
Version: 11.0
Status: UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98873
Michael Meissner changed:
What|Removed |Added
Target||powerpc64le-gnu-linux
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98870
Michael Meissner changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98519
--- Comment #21 from Michael Meissner ---
I have patches that fix the problem in the hook.
My original idea of not allowing prefixed insns in the hook doesn't work
because when the hook is called, it only sees pseudo registers.
So what my
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98519
Michael Meissner changed:
What|Removed |Added
Status|NEW |ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101019
Bug ID: 101019
Summary: GCC should consider using PLI/SLDI/PADDI to load up
64-bit constants on power10
Product: gcc
Version: 12.0
Status: UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101019
Michael Meissner changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever confirmed|0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101019
Michael Meissner changed:
What|Removed |Added
Status|NEW |ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101002
Bug ID: 101002
Summary: Some powerpc tests fail with -mlong-double-64
Product: gcc
Version: 12.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100170
Michael Meissner changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |meissner at gcc dot
gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100712
Bug ID: 100712
Summary: The vec_splatid instruction allows the creation of
XXSPLTIDP instructions which produces undefined
results.
Product: gcc
Version: 12.0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100809
--- Comment #4 from Michael Meissner ---
Note, in looking at Carl's patch, it is only for adding the built-ins. I don't
believe it adds direct support for {,u}divti3 and {,u}moddti3 to implement
these for normal __int128 variables.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100809
Michael Meissner changed:
What|Removed |Added
Last reconfirmed||2021-06-01
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96762
Michael Meissner changed:
What|Removed |Added
Last reconfirmed||2021-05-27
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96762
--- Comment #2 from Michael Meissner ---
It is in the memcpy expansion, when the compiler tries to generate lxvl and
stxvl to do the move. Unfortunately, the pattern in the expansion does a
DImode shift left by 56 to get the value into the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100809
Michael Meissner changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |meissner at gcc dot
gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100168
Michael Meissner changed:
What|Removed |Added
Ever confirmed|0 |1
Status|UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99293
Michael Meissner changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |meissner at gcc dot
gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99293
--- Comment #3 from Michael Meissner ---
Created attachment 50947
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=50947=edit
Proposed patch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=33699
Michael Meissner changed:
What|Removed |Added
CC||meissner at gcc dot gnu.org
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100327
Michael Meissner changed:
What|Removed |Added
CC||meissner at gcc dot gnu.org
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100327
Michael Meissner changed:
What|Removed |Added
Attachment #50708|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100327
--- Comment #5 from Michael Meissner ---
Unfortunately the patch does not work because there aren't suffixes for IFmode
and KFmode.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100327
Michael Meissner changed:
What|Removed |Added
Resolution|--- |FIXED
Status|ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100166
Michael Meissner changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99921
Michael Meissner changed:
What|Removed |Added
Last reconfirmed||2021-05-04
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100167
Michael Meissner changed:
What|Removed |Added
Ever confirmed|0 |1
Status|UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98952
Michael Meissner changed:
What|Removed |Added
Resolution|--- |FIXED
Status|ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98519
Michael Meissner changed:
What|Removed |Added
Status|ASSIGNED|NEW
Assignee|meissner at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99921
Bug ID: 99921
Summary: PowerPC xxeval has the wrong predicates
Product: gcc
Version: 11.0
Status: UNCONFIRMED
Severity: minor
Priority: P3
Component: target
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100166
--- Comment #2 from Michael Meissner ---
gcc.target/powerpc/lvsl-lvsr.c is another test that needs prefixed load/store
support added.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100170
Bug ID: 100170
Summary: Gcc tests gcc.target/powerpc/ppc-{eq,ne}0-1.c fail on
Power10
Product: gcc
Version: 11.0
Status: UNCONFIRMED
Severity: normal
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100166
Bug ID: 100166
Summary: Some vold-vec-{load,store} tests fail when built with
compiler configured with --with-cpu=power10
Product: gcc
Version: 11.0
Status: UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100167
Bug ID: 100167
Summary: GCC configured for power10 fails the
gcc.target/powerpc/fold-vec-div-longlong.c test
Product: gcc
Version: 11.0
Status: UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100166
--- Comment #1 from Michael Meissner ---
The test gcc.dg/vect/costmodel/ppc/costmodel-bb-slp-9a-pr63175.c also fails
because it needs to add prefixed instruction support.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100168
Bug ID: 100168
Summary: Test gcc.dg/pr56727-2.c fails on power10 code
generation
Product: gcc
Version: 11.0
Status: UNCONFIRMED
Severity: normal
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100169
Bug ID: 100169
Summary: Test gcc.dg/sms-10.c fails on power10
Product: gcc
Version: 11.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: testsuite
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98952
Michael Meissner changed:
What|Removed |Added
Ever confirmed|0 |1
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98519
--- Comment #23 from Michael Meissner ---
Obviously one approach is to use the recog_data.is_asm field to determine if
the %m constraint is in an asm and restrict it to non-prefixed memory
addresses.
However, this doesn't work, because is_asm
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98519
--- Comment #24 from Michael Meissner ---
Obviously I had a small typo in the previous example (using %U0%X0 instead of
%U1%X1) which did not matter, but here is the corrected example:
static int x;
int *p_x =
int get (void)
{
int a;
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98519
--- Comment #25 from Michael Meissner ---
Created attachment 50201
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=50201=edit
Example code for both input and output %m usage
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99133
--- Comment #1 from Michael Meissner ---
Note, the comment should read:
Note unlike the load/store/paddi instructions, these prefixed instructions do
NOT have a 'p' prefix, which means the code in rs6000_final_prescan_insn will
have to be
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99133
Bug ID: 99133
Summary: Power10 xxspltiw, xxspltidp, xxsplti32dx instructions
need to be marked as prefixed
Product: gcc
Version: 11.0
Status: UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99133
Michael Meissner changed:
What|Removed |Added
Ever confirmed|0 |1
Status|UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102059
Michael Meissner changed:
What|Removed |Added
CC||meissner at gcc dot gnu.org
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100809
Michael Meissner changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101153
Michael Meissner changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100168
Michael Meissner changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100167
Michael Meissner changed:
What|Removed |Added
Resolution|--- |FIXED
Status|ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100166
Michael Meissner changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100170
Michael Meissner changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103387
Michael Meissner changed:
What|Removed |Added
Severity|normal |major
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103317
Michael Meissner changed:
What|Removed |Added
Severity|normal |major
Priority|P3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103320
Bug ID: 103320
Summary: Spec 2017 benchmark roms_r fails on PowerPC for -Ofast
Product: gcc
Version: 12.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103317
Michael Meissner changed:
What|Removed |Added
Priority|P2 |P1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103320
Michael Meissner changed:
What|Removed |Added
Priority|P2 |P1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103318
Michael Meissner changed:
What|Removed |Added
Priority|P2 |P1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103317
Bug ID: 103317
Summary: Spec 2017 benchmark blender_r fails with -Ofast on
PowerPc (power9, power10)
Product: gcc
Version: 12.0
Status: UNCONFIRMED
Severity:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103318
Bug ID: 103318
Summary: Spec 2017 benchmark perlbench_r fails on PowerPC for
-Ofast and -O3, passes with -O2
Product: gcc
Version: 12.0
Status: UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103320
Michael Meissner changed:
What|Removed |Added
CC||bergner at gcc dot gnu.org,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103318
Michael Meissner changed:
What|Removed |Added
Priority|P3 |P2
Severity|normal
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103498
Bug ID: 103498
Summary: Spec 2017 imagick_r is 2.62% slower on Power10 with
pc-relative addressing compared to not using
pc-relative addressing
Product: gcc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=26163
Bug 26163 depends on bug 103320, which changed state.
Bug 103320 Summary: 12 Regression] Spec 2017 benchmark roms_r fails on PowerPC
for -Ofast
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103320
What|Removed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103320
Michael Meissner changed:
What|Removed |Added
Resolution|--- |WONTFIX
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99921
Michael Meissner changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102935
Michael Meissner changed:
What|Removed |Added
Resolution|--- |FIXED
Status|ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99197
Michael Meissner changed:
What|Removed |Added
CC||meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104253
Michael Meissner changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104868
--- Comment #8 from Michael Meissner ---
Matheus, try the patch I just attached to the PR that I posted to the
gcc-patches mailing list.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104868
Michael Meissner changed:
What|Removed |Added
CC||meissner at gcc dot gnu.org
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104868
--- Comment #4 from Michael Meissner ---
In looking at it, the reason is the convert from DImode to TImode has several
constraints. The constraint that matters in this case has the output being an
Altivec register, while the input is a GPR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96983
Michael Meissner changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104698
Bug ID: 104698
Summary: Inefficient code for DI to TI sign extend on power10
Product: gcc
Version: 11.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104698
--- Comment #3 from Michael Meissner ---
It goes beyond 'just use RTL'.
The problem is the code only generates an altivec instruction. So if the
__int128_t value is in a GPR, the compiler will need to do a move to the vector
registers (1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104256
--- Comment #1 from Michael Meissner ---
Created attachment 52463
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=52463=edit
Proposed patch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104256
Michael Meissner changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |meissner at gcc dot
gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104256
Michael Meissner changed:
What|Removed |Added
Resolution|--- |DUPLICATE
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104335
Michael Meissner changed:
What|Removed |Added
CC||asolokha at gmx dot com
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104253
Michael Meissner changed:
What|Removed |Added
Status|NEW |ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104124
--- Comment #3 from Michael Meissner ---
There are two things going on.
1) There is no vspltisd instruction, so we can't generate a single instruction
to load constants other than 0 or -1. Unfortunately, this was not added in
either power9 or
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104253
--- Comment #4 from Michael Meissner ---
Created attachment 52306
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=52306=edit
Patch to use the correct names for __ibm128 converts if long double is IEEE
128-bit
The problem was internally
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104253
--- Comment #8 from Michael Meissner ---
Yes, you are right. I didn't remember which functions were generated by the
compiler, but I just did all of the conversion functions.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104253
--- Comment #11 from Michael Meissner ---
The patch has been posted, I'm awaiting approval.
https://gcc.gnu.org/pipermail/gcc-patches/2022-January/589469.html
BTW, the copy_to_mode_reg bug I mentioned earlier goes away with the patch.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102059
--- Comment #31 from Michael Meissner ---
Created attachment 52383
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=52383=edit
Simpler patch to fix the problem with power8-fusion.
This patch just ignores the -mpower8-fusion option in the
1 - 100 of 145 matches
Mail list logo
