https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #50 from ard.biesheuvel at linaro dot org ---
(In reply to Ramana Radhakrishnan from comment #49)
> (In reply to ard.biesheuvel from comment #48)
> > (In reply to Richard Earnshaw from comment #47)
> > > (In reply to ard.biesheuvel fro
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #49 from Ramana Radhakrishnan ---
(In reply to ard.biesheuvel from comment #48)
> (In reply to Richard Earnshaw from comment #47)
> > (In reply to ard.biesheuvel from comment #46)
> > > One issue that this causes, which I did not see
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #48 from ard.biesheuvel at linaro dot org ---
(In reply to Richard Earnshaw from comment #47)
> (In reply to ard.biesheuvel from comment #46)
> > One issue that this causes, which I did not see mentioned anywhere in the
> > thread, is
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #47 from Richard Earnshaw ---
(In reply to ard.biesheuvel from comment #46)
> One issue that this causes, which I did not see mentioned anywhere in the
> thread, is that the use of adrp/add and adrp/ldr imposes a 4 KB section
> alignm
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
ard.biesheuvel at linaro dot org changed:
What|Removed |Added
CC||ard.biesheuvel at linar
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
Ramana Radhakrishnan changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #44 from Ramana Radhakrishnan ---
(In reply to Christophe Lyon from comment #39)
> We have backported r227748, 229160 and 229161 to our linaro-gcc-5 branch,
> and we got a bug report from the kernel team.
Sorry about the breakage.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #43 from Christophe Lyon ---
Indeed, that seems safer.
However, reading http://www.spinics.net/lists/arm-kernel/msg445346.html there
is a comment saying:
--
Note that the kernel itself must be linked with a version
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #42 from cbaylis at gcc dot gnu.org ---
While we're suggesting fixes to the kernel, wouldn't it be better if
CONFIG_ARM64_ERRATUM_843419 forced the kernel to be built with the linker
workarounds if the kernel is configured
KBUILD_LDF
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #41 from Christophe Lyon ---
Indeed, having:
ifeq ($(CONFIG_ARM64_ERRATUM_843419), y)
KBUILD_CFLAGS_MODULE+= -mcmodel=large
KBUILD_CFLAGS_MODULE+= $(call cc-option, -mpc-relative-literal-loads)
endif
in arch/arm64/Makefile d
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
cbaylis at gcc dot gnu.org changed:
What|Removed |Added
CC||cbaylis at gcc dot gnu.org
-
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #39 from Christophe Lyon ---
We have backported r227748, 229160 and 229161 to our linaro-gcc-5 branch, and
we got a bug report from the kernel team.
Indeed, when the kernel is configured with CONFIG_ARM64_ERRATUM_843419, the
support
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #38 from PeteVine ---
Hi all,
I'm trying to narrow down a similar issue on armv7 and would welcome any
suggestions where to start:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69082
Thx
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #37 from Evandro ---
Here's what I had in mind:
https://gcc.gnu.org/ml/gcc-patches/2015-11/msg01787.html
Feedback is welcome.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #36 from Evandro ---
(In reply to Ramana Radhakrishnan from comment #35)
> (In reply to Evandro from comment #32)
> > Because of side effects of the Haiffa scheduler, the loads now pile up, and
> > the ADRPs may affect the load issue
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #35 from Ramana Radhakrishnan ---
(In reply to Evandro from comment #32)
> (In reply to Ramana Radhakrishnan from comment #31)
> > (In reply to Evandro from comment #30)
> > > The performance impact of always referring to constants as
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #34 from Evandro ---
(In reply to Wilco from comment #33)
> (In reply to Evandro from comment #32)
> ADRP latency to load-address should be zero on any OoO core - ADRP is
> basically a move-immediate, so can execute early and hide any
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #33 from Wilco ---
(In reply to Evandro from comment #32)
> (In reply to Ramana Radhakrishnan from comment #31)
> > (In reply to Evandro from comment #30)
> > > The performance impact of always referring to constants as if they were
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #32 from Evandro ---
(In reply to Ramana Radhakrishnan from comment #31)
> (In reply to Evandro from comment #30)
> > The performance impact of always referring to constants as if they were far
> > away is significant on targets which
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #31 from Ramana Radhakrishnan ---
(In reply to Evandro from comment #30)
> The performance impact of always referring to constants as if they were far
> away is significant on targets which do not fuse ADRP and LDR together.
What ha
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #30 from Evandro ---
The performance impact of always referring to constants as if they were far
away is significant on targets which do not fuse ADRP and LDR together. What's
the status of the solution that evaluates the function si
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #29 from Ramana Radhakrishnan ---
Author: ramana
Date: Thu Oct 22 04:26:50 2015
New Revision: 229160
URL: https://gcc.gnu.org/viewcvs?rev=229160&root=gcc&view=rev
Log:
[Patch AArch64 63304] Fix issue with global state.
Jiong pointe
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #28 from Ramana Radhakrishnan ---
Author: ramana
Revision: 228644
Modified property: svn:log
Modified: svn:log at Fri Oct 9 11:08:05 2015
--
--- svn:log (or
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #27 from Ramana Radhakrishnan ---
(In reply to Ramana Radhakrishnan from comment #26)
> Author: ramana
> Date: Fri Oct 9 10:58:06 2015
> New Revision: 228644
>
> URL: https://gcc.gnu.org/viewcvs?rev=228644&root=gcc&view=rev
> Log:
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #26 from Ramana Radhakrishnan ---
Author: ramana
Date: Fri Oct 9 10:58:06 2015
New Revision: 228644
URL: https://gcc.gnu.org/viewcvs?rev=228644&root=gcc&view=rev
Log:
[AArch64] Handle literal pools for functions > 1 MiB in size.
T
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #25 from Ramana Radhakrishnan ---
Author: ramana
Date: Mon Sep 14 13:16:59 2015
New Revision: 227748
URL: https://gcc.gnu.org/viewcvs?rev=227748&root=gcc&view=rev
Log:
[AArch64] Handle literal pools for functions > 1 MiB in size.
T
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #24 from Ramana Radhakrishnan ---
Author: ramana
Date: Fri Sep 11 09:44:26 2015
New Revision: 227679
URL: https://gcc.gnu.org/viewcvs?rev=227679&root=gcc&view=rev
Log:
Remove separate movtf pattern - Use an iterator for all FP modes.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #23 from David Abdurachmanov
---
GCC trunk
r226676
or 15af172f2a0ea281969e3105da9f9bb100097c7d from
git://gcc.gnu.org/git/gcc.git
Date: Thu Aug 6 14:26:18 2015 +)
Rebased and applied:
https://gcc.gnu.org/ml/gcc-patche
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #22 from Ramana Radhakrishnan ---
(In reply to David Abdurachmanov from comment #21)
> I am on vacations now, but I already marked this on my TODO list. Once I
> find a free time slot I will give it a spin. I will try to report in a f
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #21 from David Abdurachmanov
---
I am on vacations now, but I already marked this on my TODO list. Once I find a
free time slot I will give it a spin. I will try to report in a few days.
BTW, I will also show up at GNU Tools Cauldro
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #20 from Ramana Radhakrishnan ---
(In reply to Ramana Radhakrishnan from comment #19)
> (In reply to Ramana Radhakrishnan from comment #18)
> > I'm taking a look into this.
>
> RFC here - https://gcc.gnu.org/ml/gcc-patches/2015-07/ms
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #19 from Ramana Radhakrishnan ---
(In reply to Ramana Radhakrishnan from comment #18)
> I'm taking a look into this.
RFC here - https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02258.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
Ramana Radhakrishnan changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
Wilco changed:
What|Removed |Added
CC||wdijkstr at arm dot com
--- Comment #17 from Wil
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
Jiong Wang changed:
What|Removed |Added
CC||jiwang at gcc dot gnu.org
--- Comment #16 f
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
Andrew Pinski changed:
What|Removed |Added
Assignee|pinskia at gcc dot gnu.org |unassigned at gcc dot
gnu.org
--
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #14 from David Abdurachmanov
---
I hit another two cases of this.
1. g2root tool, which converts GEANT geometry to ROOT geometry. It create a
single function, which contains lots of descriptions of material, shapes, etc.
all describ
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #13 from David Abdurachmanov
---
Created attachment 34416
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34416&action=edit
31-lines, minimal testcase
I am adding 31-lines minimal testcase. Should be good enough for GCC testsui
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #12 from David Abdurachmanov
---
I decided to re-enable -Os for OpenLoops. Then use powerful hardware with
32-physical-cores (x86_64) and 0.5TB of RAM to see if I could get lucky. Fired
up QEMU user mode with Fedora for AArch64 chroo
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
David Abdurachmanov changed:
What|Removed |Added
CC||david.abdurachmanov at gmail
dot c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
Andrew Pinski changed:
What|Removed |Added
Keywords||assemble-failure
Severity|mino
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #9 from Andrew Pinski ---
Actually this is much worse than what is mentioned here. Having the constant
pool be part of the .text section really does not work if the alignment of the
constants are smaller than 4 byte aligned. The ass
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
Andrew Pinski changed:
What|Removed |Added
Status|RESOLVED|NEW
Resolution|WONTFIX
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
Venkataramanan changed:
What|Removed |Added
Status|NEW |RESOLVED
CC|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
Richard Earnshaw changed:
What|Removed |Added
Priority|P3 |P5
Status|RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #5 from Venkataramanan ---
We got inspired by this bug.
https://bugs.linaro.org/show_bug.cgi?id=400
It happens at -O0 now.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
Richard Earnshaw changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
Andrew Pinski changed:
What|Removed |Added
Target||aarch64
Status|UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #2 from Venkataramanan ---
Marcus, can you please assign it to me if it is confirmed.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63304
--- Comment #1 from Venkataramanan ---
Created attachment 33515
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33515&action=edit
Attached test case
50 matches
Mail list logo
