http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53110
Bug #: 53110
Summary: GCC-4.7 generates stupid x86_64 asm
Classification: Unclassified
Product: gcc
Version: 4.7.1
Status: UNCONFIRMED
Severity: normal
Priority: P3
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53110
--- Comment #2 from peterz at infradead dot org 2012-04-25 11:11:19 UTC ---
I'll have to let Linus and Peter Anvin argue this (they're on CC), this report
is
the direct result of their complaints:
"If you can *ever* get gcc to gene
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53110
--- Comment #5 from peterz at infradead dot org 2012-04-25 12:47:31 UTC ---
Yes that's what it computes.. and no the function won't ever get used on
x86_64, but I ran it through the compiler anyway :-)
Thing is we need u128 to work on
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53110
--- Comment #6 from peterz at infradead dot org 2012-04-25 13:00:47 UTC ---
OK rectification, that's what it _should_ compute, I just noticed
add_u128() is missing: a.hi += b.hi; Anyway, that's all besides the point, the
issue i
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53110
--- Comment #8 from peterz at infradead dot org 2012-04-25 13:15:56 UTC ---
Jakub's patch seems to improve the situation:
--- gcc-bug-4.7.s 2012-04-25 14:58:21.494815266 +0200
+++ gcc-bug-4.7+.s 2012-04-25 15:14:13.784243427
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104816
--- Comment #6 from peterz at infradead dot org ---
(In reply to H.J. Lu from comment #5)
> (In reply to Andrew Cooper from comment #4)
> > I've worked around this in Xen with:
> > https://xenbits.xen.org/gitweb/?p=xen.git
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105160
peterz at infradead dot org changed:
What|Removed |Added
CC||peterz at infradead dot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105160
--- Comment #7 from peterz at infradead dot org ---
(In reply to peterz from comment #6)
> Happy accident; I've been wanting to allow doing something like:
>
> static __always_inline __pure bool arch_static_branch(struct static
: unassigned at gcc dot gnu.org
Reporter: peterz at infradead dot org
Target Milestone: ---
Please implement -fsanitize=kcfi to match llvm/clang:
https://github.com/samitolvanen/llvm-project/commit/f7bf6a87c4fd945800115a17b8b61390541fabd0
The Linux kernel patches are queued and
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102952
--- Comment #25 from peterz at infradead dot org ---
(In reply to H.J. Lu from comment #24)
> Should I submit the current patches?
Yes, I'd say so. Once merged I'll send a kernel patch to use
-mindirect-branch-cs-prefix for all RET
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102953
peterz at infradead dot org changed:
What|Removed |Added
CC||peterz at infradead dot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102952
peterz at infradead dot org changed:
What|Removed |Added
CC||peterz at infradead dot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102952
--- Comment #9 from peterz at infradead dot org ---
Created attachment 51683
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=51683&action=edit
kernel patch to test -mharden-sls=all
$ make O=defconfig CC=gcc-12.0.0 arch/x86/entry/c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102952
--- Comment #10 from peterz at infradead dot org ---
(In reply to H.J. Lu from comment #4)
> Created attachment 51679 [details]
> A patch to add -mindirect-branch-cs-prefix
>
> It adds CS prefix to call and jmp to thunk when convert
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102952
--- Comment #13 from peterz at infradead dot org ---
(In reply to H.J. Lu from comment #12)
> (In reply to peterz from comment #9)
> > Created attachment 51683 [details]
> > kernel patch to test -mharden-sls=all
> >
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102952
--- Comment #20 from peterz at infradead dot org ---
(In reply to H.J. Lu from comment #19)
> Created attachment 51685 [details]
> The v4 patch to add -mharden-sls=
That looks to do the right thing! Let me go write more validation st
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102952
--- Comment #21 from peterz at infradead dot org ---
(In reply to H.J. Lu from comment #19)
> Created attachment 51685 [details]
> The v4 patch to add -mharden-sls=
I seem to have found one 'funny':
kernel/bpf/core.o:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104816
--- Comment #12 from peterz at infradead dot org ---
On Tue, May 24, 2022 at 04:06:08PM +, cvs-commit at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104816
>
> --- Comment #11 from CVS Commits ---
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=119279
--- Comment #8 from peterz at infradead dot org ---
On Fri, Mar 14, 2025 at 08:44:19AM +, torva...@linux-foundation.org wrote:
> Now, I agree that the kernel could likely solve issues by just "don't check so
> much, then
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=119279
--- Comment #10 from peterz at infradead dot org ---
On Fri, Mar 14, 2025 at 09:41:07AM +, jakub at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=119279
>
> --- Comment #9 from Jakub Jelinek ---
>
20 matches
Mail list logo
