https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112296
--- Comment #9 from Richard Henderson ---
> Thanks. So yes,
>
> macro(x++);
>
> incrementing x twice would have been odd - but that's the usual bug
> in this kind of macro definition. Fixing it by throwing away
> side-effects (and always
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112296
--- Comment #7 from Richard Henderson ---
(In reply to Richard Biener from comment #5)
> int bad1(void) { return __builtin_constant_p(global++); }
...
> Joseph, Richard, do you have anything to add or remember discussions about
> this semantic
Priority: P3
Component: ipa
Assignee: unassigned at gcc dot gnu.org
Reporter: rth at gcc dot gnu.org
CC: marxin at gcc dot gnu.org
Target Milestone: ---
The noinline attribute affects decisions made by ipa-split.cc
and ipa-icf.cc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107389
Richard Henderson changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Summary|Alignment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107389
--- Comment #3 from Richard Henderson ---
If __builtin_assume_aligned were to work at -O0,
that would also work for me. Better, probably,
than fiddling with __attribute__((aligned)).
Assignee: unassigned at gcc dot gnu.org
Reporter: rth at gcc dot gnu.org
Target Milestone: ---
Consider
typedef __uint128_t aligned_type __attribute__((aligned(16)));
_Static_assert(__alignof(aligned_type) == 16);
__uint128_t foo(aligned_type *p) { return __atomic_load_n(p, 0
Priority: P3
Component: c
Assignee: unassigned at gcc dot gnu.org
Reporter: rth at gcc dot gnu.org
Target Milestone: ---
For a testcase such as
enum E { l = -1, z = 0, g = 1 };
int foo(void);
enum E foo(void) { return z; }
If the implementation type of 'enum E
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99696
--- Comment #1 from Richard Henderson ---
Actually, I can reproduce this with gcc 9.3 as well.
My upstream bug report simply uses gcc 11, so I assumed.
-end
Assignee: unassigned at gcc dot gnu.org
Reporter: rth at gcc dot gnu.org
Target Milestone: ---
The following is a c-ish version of
const int y = init();
which no longer works with gcc 11.
The intended advantage to the program from which this is
extracted
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97323
--- Comment #10 from Richard Henderson ---
Created attachment 49473
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49473=edit
rfc patch
The following fixes the ICE.
It seems like a hack, done at the wrong level.
Should we have in fact
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97323
--- Comment #9 from Richard Henderson ---
As a data point, this problem can be seen with any
strict-alignment target -- e.g. sparc.
Priority: P3
Component: middle-end
Assignee: unassigned at gcc dot gnu.org
Reporter: rth at gcc dot gnu.org
Target Milestone: ---
unsigned f(unsigned short x) { return (x > 0xff ? 0xff : x) & 0xff; }
cmpw$255, %di
movl$25
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94174
--- Comment #2 from Richard Henderson ---
Case 3:
void test3(__int128 a, unsigned long l)
{
if ((__int128_t)a - l <= 1)
doit();
}
currently generates as
subsx0, x0, x2
sbc x1, x1, xzr
cmp x1, 0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94174
Richard Henderson changed:
What|Removed |Added
Summary|__builtin_add_overflow vs |Missed ccmp optimizations
Assignee: unassigned at gcc dot gnu.org
Reporter: rth at gcc dot gnu.org
Target Milestone: ---
(1) Case 1:
void doit(void);
void test(unsigned long a, unsigned long l)
{
if (!__builtin_add_overflow(a, 8 - 1, ) && a <= l)
doit();
}
currently generates as
Component: target
Assignee: unassigned at gcc dot gnu.org
Reporter: rth at gcc dot gnu.org
Target Milestone: ---
We should pattern-match multiple logical operations
into the ternary logical operator. While there are
lots of obscure combinations available, probably the
most useful
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92902
--- Comment #14 from Richard Henderson ---
The only reason I can think for jump tables to be put into the text
section is the old aout format, which didn't have a separate read
only data section. There should be no reason to do that these days.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68543
Richard Henderson changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91833
--- Comment #6 from Richard Henderson ---
Author: rth
Date: Wed Sep 25 22:51:55 2019
New Revision: 276134
URL: https://gcc.gnu.org/viewcvs?rev=276134=gcc=rev
Log:
aarch64: Configure for sys/auxv.h in libgcc for lse-init.c
PR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91834
--- Comment #5 from Richard Henderson ---
Author: rth
Date: Wed Sep 25 21:48:41 2019
New Revision: 276133
URL: https://gcc.gnu.org/viewcvs?rev=276133=gcc=rev
Log:
aarch64: Fix store-exclusive in load-operate LSE helpers
PR target/91834
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91833
--- Comment #5 from Richard Henderson ---
Ah, I've been using the old time one tree build.
I'll try building aarch64-elf in pieces as I fix.
|unassigned at gcc dot gnu.org |rth at gcc dot gnu.org
--- Comment #2 from Richard Henderson ---
Hmm. When I built aarch64-elf for newlib, configure automatically
sets inhibit_libc, which avoids the whole issue.
While a test vs __GLIBC__ might work, a configure test vs sys/auxv.h
is probably better
|unassigned at gcc dot gnu.org |rth at gcc dot gnu.org
--- Comment #4 from Richard Henderson ---
Ack.
Priority: P3
Component: c
Assignee: unassigned at gcc dot gnu.org
Reporter: rth at gcc dot gnu.org
Target Milestone: ---
/* Header file */
extern int bar;
/* Source file */
static int foo;
extern int bar __attribute__((alias("foo")));
--
For this test cas
: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: rth at gcc dot gnu.org
Target Milestone: ---
In the following test case,
static inline unsigned deposit32(unsigned value, int start, int length,
unsigned fieldval
: target
Assignee: unassigned at gcc dot gnu.org
Reporter: rth at gcc dot gnu.org
Target Milestone: ---
typedef signed svec __attribute__((vector_size(16)));
typedef unsigned uvec __attribute__((vector_size(16)));
svec les(svec x, svec y) {
return x <= y;
}
uvec leu(uve
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86774
Richard Henderson changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86541
--- Comment #2 from Richard Henderson ---
(In reply to Richard Biener from comment #1)
> Given that we have a target pass that makes use of SSE regs for scalar
> operations I wonder if it would make more sense to attack this at the
> target
Component: target
Assignee: unassigned at gcc dot gnu.org
Reporter: rth at gcc dot gnu.org
Target Milestone: ---
In order to be more compatible across platforms, it would be
helpful if vector_size(8) was better supported for i386/x86_64.
The vast majority of the operations can
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84010
--- Comment #6 from Richard Henderson ---
For better rematerialization, I wonder if it wouldn't be better
to represent this as
(set (reg:P tmp)
(const:P (unspec [(symbol_ref "xxx")] UNSPEC_TLSIE)))
prior to reload, and split to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84010
--- Comment #3 from Richard Henderson ---
(In reply to James Clarke from comment #2)
> Here is a completely untested patch which should in theory resolve this
> series of issues. This doesn't introduce rematerialization for them (or, if
> it's
Assignee: unassigned at gcc dot gnu.org
Reporter: rth at gcc dot gnu.org
Target Milestone: ---
Created attachment 43224
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43224=edit
preprocessed input and assembly output
Seen in top-of-tree qemu:
lduw[%fp+1823],
||rth at gcc dot gnu.org
Resolution|--- |DUPLICATE
--- Comment #3 from Richard Henderson ---
Dup.
*** This bug has been marked as a duplicate of bug 80037 ***
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80037
Richard Henderson changed:
What|Removed |Added
CC||mitalis at iiitd dot ac.in
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80037
Richard Henderson changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80037
--- Comment #5 from Richard Henderson ---
Author: rth
Date: Fri May 26 19:33:19 2017
New Revision: 248526
URL: https://gcc.gnu.org/viewcvs?rev=248526=gcc=rev
Log:
PR libgcc/80037
Backport from mainline
* config/alpha/t-alpha
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80037
--- Comment #4 from Richard Henderson ---
Author: rth
Date: Fri May 26 19:29:46 2017
New Revision: 248525
URL: https://gcc.gnu.org/viewcvs?rev=248525=gcc=rev
Log:
PR libgcc/80037
Backport from mainline
* config/alpha/t-alpha
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80037
--- Comment #3 from Richard Henderson ---
Author: rth
Date: Fri May 26 18:45:59 2017
New Revision: 248522
URL: https://gcc.gnu.org/viewcvs?rev=248522=gcc=rev
Log:
PR libgcc/80037
* config/alpha/t-alpha (CRTSTUFF_T_CFLAGS): New.
Modified:
||2017-03-13
Assignee|unassigned at gcc dot gnu.org |rth at gcc dot gnu.org
Ever confirmed|0 |1
--- Comment #1 from Richard Henderson ---
Created attachment 40967
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40967=e
Assignee: unassigned at gcc dot gnu.org
Reporter: rth at gcc dot gnu.org
Target Milestone: ---
Looks similar to PR40332, but targeting alpha.
The cause is gcc writing gas directives for unwind for
__do_global_ctors_aux, and also writing the terminator
for the .eh_frame section via
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71916
--- Comment #8 from Richard Henderson ---
(gdb) call debug_cfi_row(cur_row)
.cfi_def_cfa 7, 16
.cfi_offset 3, -16
.cfi_offset 16, -8
(gdb) call debug_cfi_row(ti->beg_row)
.cfi_def_cfa 7, 8
.cfi_offset 16,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71636
--- Comment #1 from Richard Henderson ---
Oh, and I meant to mention -- if the target doesn't have an andnot
insn, both formulations are identical in complexity and minimal path.
Which might suggest *always* performing the transformation at a
Component: rtl-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: rth at gcc dot gnu.org
Target Milestone: ---
The following functions are equivalent,
unsigned f(unsigned x, unsigned b)
{
return x & ((1U << b) - 1);
}
unsigned g(unsigned x, unsigned b)
{
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69391
Richard Henderson changed:
What|Removed |Added
Status|ASSIGNED|NEW
Summary|[5/6
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60723
Bug 60723 depends on bug 61817, which changed state.
Bug 61817 Summary: Inconsistent location of tokens in the expansion list of a
built-in macro
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61817
What|Removed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61817
Richard Henderson changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61817
--- Comment #4 from Richard Henderson ---
Author: rth
Date: Wed Apr 6 18:35:16 2016
New Revision: 234794
URL: https://gcc.gnu.org/viewcvs?rev=234794=gcc=rev
Log:
PR preprocessor/61817
PR preprocessor/69391
* internal.h
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69391
--- Comment #6 from Richard Henderson ---
Author: rth
Date: Wed Apr 6 18:35:16 2016
New Revision: 234794
URL: https://gcc.gnu.org/viewcvs?rev=234794=gcc=rev
Log:
PR preprocessor/61817
PR preprocessor/69391
* internal.h
||rth at gcc dot gnu.org
Assignee|unassigned at gcc dot gnu.org |rth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70355
--- Comment #3 from Richard Henderson ---
Author: rth
Date: Tue Mar 29 15:19:00 2016
New Revision: 234524
URL: https://gcc.gnu.org/viewcvs?rev=234524=gcc=rev
Log:
PR middle-end/70355
* lower-subreg.c (simplify_subreg_concatn): Reject
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70355
Richard Henderson changed:
What|Removed |Added
Status|ASSIGNED|NEW
Summary|[5/6
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70355
--- Comment #2 from Richard Henderson ---
Created attachment 38113
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38113=edit
proposed patch
Testing the following, which works on the reduced test case.
As a missed-optimization, we really
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64971
--- Comment #8 from Richard Henderson ---
Created attachment 38112
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38112=edit
proposed patch
Andrew's approach to force the SYMBOL_REF to DImode is certainly one way
to approach it; another
||rth at gcc dot gnu.org
Assignee|unassigned at gcc dot gnu.org |rth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69845
--- Comment #7 from Richard Henderson ---
Proposed patch
https://gcc.gnu.org/ml/gcc-patches/2016-03/msg01255.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70199
Bug 70199 depends on bug 70273, which changed state.
Bug 70273 Summary: [6 regression] FAIL: g++.dg/ext/label13a.C -std=gnu++98
execution test / scan-assembler _ZN1CC4Ev
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
What
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
--- Comment #11 from Richard Henderson ---
Author: rth
Date: Mon Mar 21 23:03:56 2016
New Revision: 234386
URL: https://gcc.gnu.org/viewcvs?rev=234386=gcc=rev
Log:
PR c++/70273
* decl.c (notice_forced_label_r): New.
(cp_finish_decl): Use
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
Richard Henderson changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
||rth at gcc dot gnu.org
Assignee|unassigned at gcc dot gnu.org |rth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70199
--- Comment #6 from Richard Henderson ---
Author: rth
Date: Wed Mar 16 16:50:18 2016
New Revision: 234261
URL: https://gcc.gnu.org/viewcvs?rev=234261=gcc=rev
Log:
PR middle-end/70199
* function.h (struct function): Add
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
Richard Henderson changed:
What|Removed |Added
Attachment #38003|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70261
--- Comment #3 from Richard Henderson ---
Created attachment 37993
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37993=edit
aarch64 pbase_type_info.ii
This will ICE with just cc1plus -O.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68714
--- Comment #11 from Richard Henderson ---
Author: rth
Date: Wed Mar 16 23:53:01 2016
New Revision: 234271
URL: https://gcc.gnu.org/viewcvs?rev=234271=gcc=rev
Log:
Gimplify vec_cond_expr with condition inside
PR middle-end/70240
PR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70240
--- Comment #11 from Richard Henderson ---
Author: rth
Date: Wed Mar 16 23:53:18 2016
New Revision: 234273
URL: https://gcc.gnu.org/viewcvs?rev=234273=gcc=rev
Log:
PR middle-end/70240
* gcc.c-torture/compile/pr70240.c: New.
Added:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70240
--- Comment #10 from Richard Henderson ---
Author: rth
Date: Wed Mar 16 23:53:10 2016
New Revision: 234272
URL: https://gcc.gnu.org/viewcvs?rev=234272=gcc=rev
Log:
Revert r231575
PR middle-end/70240
PR middle-end/68215
2015-12-11 Eric
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70048
Richard Henderson changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70240
--- Comment #9 from Richard Henderson ---
Author: rth
Date: Wed Mar 16 23:53:01 2016
New Revision: 234271
URL: https://gcc.gnu.org/viewcvs?rev=234271=gcc=rev
Log:
Gimplify vec_cond_expr with condition inside
PR middle-end/70240
PR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70240
Richard Henderson changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70199
Richard Henderson changed:
What|Removed |Added
Summary|[5/6 Regression] Crash at |[5 Regression] Crash at -O2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68215
--- Comment #5 from Richard Henderson ---
Author: rth
Date: Wed Mar 16 23:53:01 2016
New Revision: 234271
URL: https://gcc.gnu.org/viewcvs?rev=234271=gcc=rev
Log:
Gimplify vec_cond_expr with condition inside
PR middle-end/70240
PR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
Richard Henderson changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |rth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70120
--- Comment #9 from Richard Henderson ---
Ah right, -ffunction-sections.
That requires a more extensive, though less hackish, fix.
Will post a new patch later this afternoon.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70048
--- Comment #23 from Richard Henderson ---
(In reply to Jiong Wang from comment #21)
> Please check the documentation at
> http://infocenter.arm.com/help/topic/com.arm.doc.uan0015b/
> Cortex_A57_Software_Optimization_Guide_external.pdf, page 14,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
--- Comment #9 from Richard Henderson ---
Created attachment 38003
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38003=edit
proposed patch
Alternately, instead of setting local_decls early (and doing
other tri-state-ish things in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68215
--- Comment #6 from Richard Henderson ---
Author: rth
Date: Wed Mar 16 23:53:10 2016
New Revision: 234272
URL: https://gcc.gnu.org/viewcvs?rev=234272=gcc=rev
Log:
Revert r231575
PR middle-end/70240
PR middle-end/68215
2015-12-11 Eric
||2016-03-17
CC||rth at gcc dot gnu.org
Ever confirmed|0 |1
--- Comment #2 from Richard Henderson ---
This also fails on aarch64 stage1 libstdc++, also during combine.
#1 0x0100dc90 in replace_rtx
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70048
--- Comment #22 from Richard Henderson ---
Author: rth
Date: Wed Mar 16 21:23:05 2016
New Revision: 234269
URL: https://gcc.gnu.org/viewcvs?rev=234269=gcc=rev
Log:
PR target/70048
* config/aarch64/aarch64.c (virt_or_elim_regno_p): New.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70048
--- Comment #19 from Richard Henderson ---
(In reply to Jiong Wang from comment #16)
> But there is a performance issue as described at
>
> https://gcc.gnu.org/ml/gcc-patches/2016-02/msg00281.html
>
> "this patch forces register scaling
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70240
Richard Henderson changed:
What|Removed |Added
CC||rguenth at gcc dot gnu.org
---
||rth at gcc dot gnu.org
Assignee|unassigned at gcc dot gnu.org |rth at gcc dot gnu.org
--- Comment #3 from Richard Henderson ---
Mine.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70120
--- Comment #6 from Richard Henderson ---
Created attachment 37975
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37975=edit
proposed patch
This is kind of a hack, but not too bad.
Zdenek, could you please test on that third testcase
||rth at gcc dot gnu.org
Assignee|unassigned at gcc dot gnu.org |rth at gcc dot gnu.org
--- Comment #5 from Richard Henderson ---
The problem here is the literal pool isn't a multiple of the
instruction size, so the Letext label is "misaligned", whi
||rth at gcc dot gnu.org
Assignee|unassigned at gcc dot gnu.org |rth at gcc dot gnu.org
--- Comment #5 from Richard Henderson ---
Mine.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68714
Richard Henderson changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68714
--- Comment #8 from Richard Henderson ---
Author: rth
Date: Mon Mar 14 20:48:15 2016
New Revision: 234196
URL: https://gcc.gnu.org/viewcvs?rev=234196=gcc=rev
Log:
PR tree-opt/68714
* tree-ssa-reassoc.c (ovce_extract_ops,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70048
--- Comment #13 from Richard Henderson ---
Created attachment 37911
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37911=edit
aggressive patch
Consider something like this, whereby we allow (sfp + scale + const)
as an address all the way
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70128
--- Comment #7 from Richard Henderson ---
(In reply to Andrew Pinski from comment #5)
> I still say this is undefined even with -fno-strict-aliasing because
> patching a function is undefined.
Oh please. I think that's short-sighted.
I don't
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70048
Richard Henderson changed:
What|Removed |Added
Attachment #37886|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70048
--- Comment #9 from Richard Henderson ---
While I fully believe in CSE'ing "base + reg*scale" when talking about
non-stack-based pointers, when it comes to stack-based data access I'm
less certain about the proper approach.
All things work out
||rth at gcc dot gnu.org
Assignee|unassigned at gcc dot gnu.org |rth at gcc dot gnu.org
--- Comment #6 from Richard Henderson ---
Created attachment 37886
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37886=edit
proposed patch
I agree -- at mini
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70061
Richard Henderson changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70061
--- Comment #4 from Richard Henderson ---
Author: rth
Date: Mon Mar 7 11:48:57 2016
New Revision: 234025
URL: https://gcc.gnu.org/viewcvs?rev=234025=gcc=rev
Log:
PR rtl-opt/70061
* tree-outofssa.c (emit_partition_copy): Flush pending stack
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70061
--- Comment #3 from Richard Henderson ---
Created attachment 37875
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37875=edit
proposed patch
Thanks, Jeff, the errant stack adjustment was a good hint.
The problem is that we are emitting
||rth at gcc dot gnu.org
Assignee|unassigned at gcc dot gnu.org |rth at gcc dot gnu.org
Component: middle-end
Assignee: unassigned at gcc dot gnu.org
Reporter: rth at gcc dot gnu.org
Target Milestone: ---
Quoting Ingo Monlar, via a LKML discussion:
=
It could be combined with the following 'safe' runtime behavior: when built
with
-Ow then all uninitialized
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70024
Richard Henderson changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70024
--- Comment #6 from Richard Henderson ---
Author: rth
Date: Thu Mar 3 01:40:29 2016
New Revision: 233926
URL: https://gcc.gnu.org/viewcvs?rev=233926=gcc=rev
Log:
PR libffi/70024
* Makefile.am (libffi_version_script): Look in cwd for
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70024
--- Comment #5 from Richard Henderson ---
Author: rth
Date: Wed Mar 2 23:28:11 2016
New Revision: 233921
URL: https://gcc.gnu.org/viewcvs?rev=233921=gcc=rev
Log:
PR libffi/70024
* Makefile.am (libffi_version_script): New.
|unassigned at gcc dot gnu.org |rth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67145
--- Comment #16 from Richard Henderson ---
Author: rth
Date: Wed Mar 2 21:09:54 2016
New Revision: 233916
URL: https://gcc.gnu.org/viewcvs?rev=233916=gcc=rev
Log:
PR rtl-opt/67145
* simplify-rtx.c (simplify_plus_minus): Allow reassoc
1 - 100 of 781 matches
Mail list logo
