https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79144
Alan Modra changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79801
Alan Modra changed:
What|Removed |Added
Severity|normal |enhancement
: rtl-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: amodra at gmail dot com
Target Milestone: ---
On powerpc, the equivalences created by add_store_equivs in ira.c seem like
they might not be very effective.
For reg1 not being assigned a hard reg, they allow lra
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77853
Alan Modra changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
||2016-10-04
CC||amodra at gmail dot com
Ever confirmed|0 |1
--- Comment #3 from Alan Modra ---
Confirmed. See also pr77853
Severity: normal
Priority: P3
Component: c
Assignee: unassigned at gcc dot gnu.org
Reporter: amodra at gmail dot com
Target Milestone: ---
int
f (int x)
{
int ret = 0;
switch (x)
{
case 1:
ret++;
/* Fall through. */
/* Important
: target
Assignee: unassigned at gcc dot gnu.org
Reporter: amodra at gmail dot com
Target Milestone: ---
Created attachment 39668
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=39668=edit
testcase
The attached testcase taken from rs6000.c output_cbranch shows a wr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77676
--- Comment #3 from Alan Modra ---
Bisect points to git commit b9833bfd, svn rev 240298.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72771
Alan Modra changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77324
--- Comment #9 from Alan Modra ---
Eric, I think the condition should be "!strict" rather than
"reload_in_progress". I'm not certain about that, but I think the patch might
fail if reload needs to iterate over insns more than once. For
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77324
Alan Modra changed:
What|Removed |Added
CC||amodra at gmail dot com
--- Comment #3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77743
Alan Modra changed:
What|Removed |Added
Status|WAITING |NEW
--- Comment #2 from Alan Modra ---
I
-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: amodra at gmail dot com
Target Milestone: ---
Found during build of current binutils.
bfd/ihex.c:222:19: error: '%03o' directive writing between 3 and 11 bytes into
a region of size 9 [-Werror=format-length=]
sprintf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78056
--- Comment #13 from Alan Modra ---
Thanks Martin, so it is HAVE_AS_POWER9 being undefined, which then results in
TARGET_P9_VECTOR and others being defined to zero. Kelvin's fix of not
defining the builtins when no power9 support is available
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78056
Alan Modra changed:
What|Removed |Added
CC||amodra at gmail dot com
--- Comment #10
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78325
Alan Modra changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
||2016-11-14
CC||amodra at gmail dot com
Assignee|unassigned at gcc dot gnu.org |amodra at gmail dot com
Ever confirmed|0 |1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78325
--- Comment #2 from Alan Modra ---
Created attachment 40040
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40040=edit
proposed fix
This hasn't been bootstrapped yet anywhere, but ought to keep pr70890 fixed and
does result in mips
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72749
Alan Modra changed:
What|Removed |Added
CC|amodra at gcc dot gnu.org |
--- Comment #12 from Alan Modra
at gcc dot gnu.org |amodra at gmail dot com
--- Comment #6 from Alan Modra ---
The problem happens in lra-contraints.c:process_address_1. Search for
HAVE_lo_sum. A high/lo_sum address is correctly rejected by the rs6000 backend
as improper for V4 pic here:
*ad.inner
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79066
Alan Modra changed:
What|Removed |Added
CC||amodra at gmail dot com
--- Comment #4
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72749
Alan Modra changed:
What|Removed |Added
CC||amodra at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72749
--- Comment #6 from Alan Modra ---
OK, currently testing TARGET_LEGITIMATE_COMBINED_INSN solution. Thanks for the
analysis!
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
Alan Modra changed:
What|Removed |Added
CC||amodra at gmail dot com
--- Comment #12
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #17 from Alan Modra ---
I see the same sort of invalid rtl sharing errors on powerpc64le with both
https://gcc.gnu.org/ml/gcc-patches/2016-11/msg03117.html and
https://gcc.gnu.org/ml/gcc-patches/2016-11/msg03116.html applied.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #14 from Alan Modra ---
I meant the last one attached, which I see is the same as the first one. ie.
just use copy_rtx and don't worry about extra rtl.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #18 from Alan Modra ---
Ditto for just https://gcc.gnu.org/ml/gcc-patches/2016-11/msg03117.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #16 from Alan Modra ---
The patch referenced in comment #15 alone, ie. without
https://gcc.gnu.org/ml/gcc-patches/2016-11/msg03117.html, results in an
--enable-checking=yes,rtl bootstrap failure on powerpc64le-linux.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78614
--- Comment #19 from Alan Modra ---
Cancel those last three comments. I wasn't actually testing with Jakub's
rs6000.c patch.
Everything now looks good with Jakub's rs6000.c and simplify-rtx.c patch, plus
the rs6000.c insn_is_swappable_p patch
Assignee: unassigned at gcc dot gnu.org
Reporter: amodra at gmail dot com
Target Milestone: ---
-static and -pie are incompatible, yet gcc neither warns nor errors on the
combination. What's more, gcc's behaviour depends on whether gcc was built
with --enable-default-pie
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80044
Alan Modra changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80044
--- Comment #1 from Alan Modra ---
Revising my original comments..
According to the gcc docs -static is supposed to just prevent linking with
dynamic libraries. I agree that this is traditional linker behaviour.
That means, for example, that
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80479
Alan Modra changed:
What|Removed |Added
CC||amodra at gmail dot com
--- Comment #9
de |
Assignee|unassigned at gcc dot gnu.org |amodra at gmail dot com
--- Comment #19 from Alan Modra ---
Removing the autoreply email from CCs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=45053
Alan Modra changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60818
--- Comment #19 from Alan Modra ---
Yes, r246294 powerpc64le-linux-gcc -O1 -misel ICEs on the last testcase. An
earlier compiler I had laying around, 7.0.0 20160616, does not.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81793
Alan Modra changed:
What|Removed |Added
Status|ASSIGNED|NEW
Assignee|amodra at gmail dot
Priority: P3
Component: inline-asm
Assignee: unassigned at gcc dot gnu.org
Reporter: amodra at gmail dot com
Target Milestone: ---
gcc doesn't have a simple way to say that a pointer passed to an inline asm is
used to address an array. "m" (*p) unfortun
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81793
--- Comment #2 from Alan Modra ---
Oh, and you're failing the link due to not having static -lc and -lm
installed..
||2017-08-11
CC||amodra at gmail dot com
Component|other |target
Assignee|unassigned at gcc dot gnu.org |amodra at gmail dot com
Ever confirmed|0 |1
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81747
Alan Modra changed:
What|Removed |Added
CC||amodra at gmail dot com
--- Comment #3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81747
--- Comment #5 from Alan Modra ---
Segher, the confusion is due to looking at the wrong BRANCH_EDGE occurrence in
cse.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81747
--- Comment #7 from Alan Modra ---
Yeah, I have more or less the same patch about to be bootstrapped.
at gcc dot gnu.org |amodra at gmail dot com
|ASSIGNED
URL||https://gcc.gnu.org/ml/gcc-
||patches/2017-08/msg01174.ht
||ml
Assignee|unassigned at gcc dot gnu.org |amodra at gmail
Priority: P3
Component: inline-asm
Assignee: unassigned at gcc dot gnu.org
Reporter: amodra at gmail dot com
Target Milestone: ---
gcc doesn't have a simple way to say that a pointer passed to an inline asm is
used to address an array. "m" (*p) unfortunately
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81890
Alan Modra changed:
What|Removed |Added
Keywords||documentation
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79286
Alan Modra changed:
What|Removed |Added
URL|https://gcc.gnu.org/ml/gcc- |https://gcc.gnu.org/ml/gcc-
Priority: P3
Component: target
Assignee: unassigned at gcc dot gnu.org
Reporter: amodra at gmail dot com
Target Milestone: ---
Configuring any powerpc linux target with --enable-default-pie gives a broken
compiler that uses crt1.o, crtbegin.o and crtend.o
||2017-06-22
CC||doko at ubuntu dot com,
||wschmidt at gcc dot gnu.org
Assignee|unassigned at gcc dot gnu.org |amodra at gmail dot com
Ever confirmed|0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67328
Alan Modra changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81193
Alan Modra changed:
What|Removed |Added
CC||amodra at gmail dot com
--- Comment #3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81193
--- Comment #6 from Alan Modra ---
> Alternatively, we would need to do two passes. On the first pass resolve all
> of the normal functions and data locations, setting the location of ifunc
> functions in the TOC to be an error function. The
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81170
Alan Modra changed:
What|Removed |Added
Keywords||patch
Priority|P3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79801
Alan Modra changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63281
Alan Modra changed:
What|Removed |Added
Attachment #33503|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63281
--- Comment #8 from Alan Modra ---
Created attachment 42187
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=42187=edit
[RS6000] Address cost
Somewhat related, costing constants properly also needs a proper cost to
loading from memory.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81996
Alan Modra changed:
What|Removed |Added
Summary|__builtin_return_address(0) |powerpc
|does not work
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81996
Alan Modra changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
at gcc dot gnu.org |amodra at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81747
Alan Modra changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81170
Alan Modra changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
||amodra at gmail dot com
Resolution|--- |FIXED
Target Milestone|7.3 |7.2
--- Comment #16 from Alan Modra ---
Fixed.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81298
Bug 81298 depends on bug 81295, which changed state.
Bug 81295 Summary: bootstrap broken on powerpc-linux-gnu with
--enable-default-pie
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81295
What|Removed |Added
: normal
Priority: P3
Component: lto
Assignee: unassigned at gcc dot gnu.org
Reporter: amodra at gmail dot com
CC: marxin at gcc dot gnu.org
Target Milestone: ---
Summary: [8 Regression] lto debugobj references __gnu_lto_slim
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82575
--- Comment #3 from Alan Modra ---
Patch now bootstrapped and regression tested powerpc64le-linux.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81890
Alan Modra changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82575
--- Comment #5 from Alan Modra ---
> --- Comment #4 from Richard Biener ---
> Hmm, but those symbols will prevail, enlarging the final symbol table?
> Or are weak + hidden symbols removed even for shared libaries?
Well, they are still
Assignee: unassigned at gcc dot gnu.org
Reporter: amodra at gmail dot com
CC: marxin at gcc dot gnu.org
Target Milestone: ---
Lack of a .note.GNU-stack section results in ld testsuite failures:
arm-linuxeabi FAIL: PR ld/12982
hppa-linux FAIL: PR ld/12982
m68k-linux
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82575
--- Comment #2 from Alan Modra ---
Created attachment 42397
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=42397=edit
proposed patch
I hadn't debugged past grepping for "debugobj" when I created the bugzilla.
Now that I've looked
||2017-10-24
CC||amodra at gmail dot com,
||marxin at gcc dot gnu.org
Component|other |lto
Ever confirmed|0 |1
--- Comment #1 from Alan
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82928
Alan Modra changed:
What|Removed |Added
Attachment #42571|0 |1
is obsolete|
Priority: P3
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: amodra at gmail dot com
Target Milestone: ---
Created attachment 42686
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=42686=edit
preprocessed source
dur
Assignee: unassigned at gcc dot gnu.org
Reporter: amodra at gmail dot com
Target Milestone: ---
Created attachment 42571
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=42571=edit
somewhat reduced testcase
Seen with current glibc "make check", gcc 20171109
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82757
Alan Modra changed:
What|Removed |Added
CC||amodra at gmail dot com
--- Comment #2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82757
--- Comment #5 from Alan Modra ---
Incidentally, the only reason ld.bfd gets past the "plugin needed" warning is
that it only warns on a common symbol called __gnu_lto_slim, not an undefined
one.. Bump st_name by one for __gnu_lto_ symbols,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82757
--- Comment #9 from Alan Modra ---
If you start off with "___gnu_lto_", there is code in simple_object_elf.c to
handle the extra '_', and now in binutils too. Skipping just one '_' will
leave you with a symbol that matches "__gnu_lto_", and
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82757
--- Comment #7 from Alan Modra ---
> Bump st_name by one for __gnu_lto_
Better make that two. Some targets will have an extra '_' prefix.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82575
Bug 82575 depends on bug 82687, which changed state.
Bug 82687 Summary: [8 regression] g++.dg/asan/default-options-1.C fails
starting with r253914
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82687
What|Removed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82687
Alan Modra changed:
What|Removed |Added
Status|NEW |RESOLVED
Blocks|
||2018-04-27
Assignee|unassigned at gcc dot gnu.org |amodra at gmail dot com
Ever confirmed|0 |1
Priority: P3
Component: middle-end
Assignee: unassigned at gcc dot gnu.org
Reporter: amodra at gmail dot com
Target Milestone: ---
Created attachment 44063
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=44063=edit
testcase
The attached testcase taken f
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82575
--- Comment #7 from Alan Modra ---
> --- Comment #6 from rguenther at suse dot de ---
> OK. I suppose they are properly prevailed by any global symbol of the same
> name
> as well? Like a weak definition with default visibility? Or is there
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82575
Alan Modra changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82598
Alan Modra changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
||2018-01-25
Assignee|unassigned at gcc dot gnu.org |amodra at gmail dot com
Ever confirmed|0 |1
--- Comment #1 from Alan Modra ---
Created attachment 43236
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43236=edit
proposed pa
Component: target
Assignee: unassigned at gcc dot gnu.org
Reporter: amodra at gmail dot com
Target Milestone: ---
Created attachment 43235
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43235=edit
testcase
The attached testcase fails on ppc64le with -O2 -mcpu=pow
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84033
Alan Modra changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84300
Alan Modra changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
at gcc dot gnu.org |amodra at gmail dot com
--- Comment #2 from Alan Modra ---
Testing what should be an obvious fix.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82368
Alan Modra changed:
What|Removed |Added
CC||amodra at gmail dot com
--- Comment #10
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84300
Alan Modra changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57717
Alan Modra changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81682
Alan Modra changed:
What|Removed |Added
CC||amodra at gmail dot com
--- Comment #2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72802
--- Comment #12 from Alan Modra ---
gcc.c-torture/compile/pr72802.c failed for me (likely with -mcpu=power9) with
the version of gcc I happened to have at the time I developed the patch in #c5.
I'm not sure now whether it was to demonstrate the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84033
--- Comment #8 from Alan Modra ---
Hi Breno, the first gcc-8 has not yet been released (current aim is for a
release mid April), nor has there been a release from the gcc-7 or gcc-6
branches containing this bug fix. I missed out on gcc-7.3 by a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83707
Alan Modra changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85075
Alan Modra changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84762
Alan Modra changed:
What|Removed |Added
CC||tamar.christina at arm dot com
--- Comment
||2018-12-27
CC|amodra at gcc dot gnu.org |
Assignee|unassigned at gcc dot gnu.org |amodra at gmail dot com
Ever confirmed|0 |1
601 - 700 of 872 matches
Mail list logo