https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79443
--- Comment #3 from Ian Lance Taylor ---
Created attachment 40708
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40708=edit
crypto patch
This patch may fix the crypto/sha256 problem.
Any other problems? `make check-target-libgo` should
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79443
--- Comment #1 from Ian Lance Taylor ---
Created attachment 40707
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40707=edit
Possible patch
Can you check whether this patch fixes the problem? Thanks.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79281
--- Comment #14 from Ian Lance Taylor ---
> Could you maybe also backport the fix for PR/79037? [1]
Done.
> Btw, even with the fixes from this PR/79281 and PR/79037, the "go" command is
> still crashing on m68k with gcc-6. It might be
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79281
--- Comment #11 from Ian Lance Taylor ---
Thanks. I committed the patch to the GCC 6 branch.
GCC 7 will require a different fix, as the code has moved from C to Go. I'm
not sure what the best approach is.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79281
--- Comment #7 from Ian Lance Taylor ---
It sounds like you have a patch for GCC 6. If you send it in I can apply it.
The error you show must be from `make -j`, as compiling a file in libgfortran
would not invoke go1. What is the actual
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79289
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79289
Ian Lance Taylor changed:
What|Removed |Added
CC||aoliva at gcc dot gnu.org,
Severity: normal
Priority: P3
Component: debug
Assignee: unassigned at gcc dot gnu.org
Reporter: ian at airs dot com
Target Milestone: ---
Compile this program with current mainline with -g on GNU/Linux:
static void sub() {}
void f1() { sub(); }
void f2(void
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79281
--- Comment #2 from Ian Lance Taylor ---
Perhaps this would be fixed by adding __attribute__((aligned(4))) to some field
of struct Sched in proc.c.
This code is all different on trunk/GCC7. I don't know whether it still has
the same problem or
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79037
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79037
--- Comment #9 from Ian Lance Taylor ---
The backend.h interface says that if you set the alignment, that is the
alignment. I could change it to the GCC version--you can only increase the
alignment--but I'd rather keep the backend.h interface
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79146
--- Comment #5 from Ian Lance Taylor ---
Well, it helped in that it uncovered a different problem.
Fixed again.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79037
--- Comment #7 from Ian Lance Taylor ---
Can someone with m68k hardware please test the patch at
https://golang.org/cl/35478? Thanks. (To download just the patch as a zip
file:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79037
--- Comment #6 from Ian Lance Taylor ---
I don't think it's the type descriptors that need to be aligned, I think it's
just the GC symbol pointers. Those are the ones whose names end in "$gc" in
the list above.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79146
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79122
--- Comment #2 from Ian Lance Taylor ---
Note that the race detector doesn't work at all with gccgo. So while this is a
pretty bad error message, the only fix for this bug is going to be to produce a
better error message.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79068
Ian Lance Taylor changed:
What|Removed |Added
CC||ian at airs dot com
--- Comment #4
||ian at airs dot com
Resolution|--- |FIXED
--- Comment #6 from Ian Lance Taylor ---
Should be fixed.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78789
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78978
--- Comment #1 from Ian Lance Taylor ---
If the libstdc++ approach works and is acceptable, it seems to me we should do
the same for libgo.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78763
--- Comment #8 from Ian Lance Taylor ---
Here is a test case that recreates the problem.
package p
import (
"unsafe"
)
func F() int {
if unsafe.Sizeof(0) == 8 {
return 8
}
return 0
}
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78431
Bug 78431 depends on bug 78432, which changed state.
Bug 78432 Summary: [7 Regression] -fdump-go-spec ICEs for aligned causing x32
libgo library to fail to build
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78432
What|Removed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78431
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78432
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77910
Ian Lance Taylor changed:
What|Removed |Added
Resolution|WONTFIX |FIXED
--- Comment #9 from Ian Lance
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77910
--- Comment #7 from Ian Lance Taylor ---
Coincidentally I am going to submit the patch to fix this today. It was held
up for a while in code review on the master repository. (The fix in the master
repository is https://golang.org/cl/33295).
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78172
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78172
--- Comment #9 from Ian Lance Taylor ---
Regarding arpcom, the syscall package is not really intended to include
everything that seems useful. That is the goal of the external package
golang.org/x/sys/unix. The syscall package only includes
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78172
--- Comment #7 from Ian Lance Taylor ---
Thanks. Does a patch like this fix the immediate ucred problem?
diff --git a/libgo/mksysinfo.sh b/libgo/mksysinfo.sh
index 3640476..6c457b6 100755
--- a/libgo/mksysinfo.sh
+++ b/libgo/mksysinfo.sh
@@
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78172
--- Comment #5 from Ian Lance Taylor ---
Thanks. I do not see any problem in that file you attached. I also don't see
anything like the line in the original error message.
Looking more closely at that line, I see now that it appears to have
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78145
Ian Lance Taylor changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78172
--- Comment #3 from Ian Lance Taylor ---
If that is the complete line, then something has gone badly wrong.
Can you get a copy of the complete generated sysinfo.go file?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78172
--- Comment #1 from Ian Lance Taylor ---
What does that whole line look like? It seems to be truncated in the output
for some reason.
Is _priv_t defined elsewhere in the file? That is, is there a line that starts
with "type _priv_t "? If
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78144
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78143
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77977
Ian Lance Taylor changed:
What|Removed |Added
CC||jakub at redhat dot com
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77910
--- Comment #1 from Ian Lance Taylor ---
Works for me.
Do you have a copy of the gc toolchain on your computer? Is the environment
variable GOROOT set?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77861
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=46986
Ian Lance Taylor changed:
What|Removed |Added
CC||socnet.delphin at laposte dot
net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77809
Ian Lance Taylor changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77780
--- Comment #3 from Ian Lance Taylor ---
My inclination would be to overhaul all the identifiers in the gofrontend to
rationalize them from the rather random collection they are now.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77688
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
||ian at airs dot com
Resolution|--- |FIXED
--- Comment #3 from Ian Lance Taylor ---
Fixed. Thanks for noticing.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77625
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77642
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
||ian at airs dot com
Resolution|--- |WONTFIX
--- Comment #12 from Ian Lance Taylor ---
I don't think there is anything to fix here, so I'm going to close this. GCC
is not the master source for libffi.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70597
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72814
Ian Lance Taylor changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72814
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72812
Ian Lance Taylor changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72812
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71697
--- Comment #3 from Ian Lance Taylor ---
The running the failing command yourself with the -v option and see which
options are being passed to the linker.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71697
--- Comment #1 from Ian Lance Taylor ---
Note that as far as I can tell that error is only emitted by the GNU linker
when the --no-add-needed option is passed to the linker. Is something on your
system adding that option or making it the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71692
--- Comment #2 from Ian Lance Taylor ---
The relevant failures are:
--- FAIL: TestCloneNEWUSERAndRemapNoRootDisableSetgroups (0.00s)
exec_linux_test.go:74: Cmd failed with err fork/exec /usr/bin/whoami:
in
valid argument, output:
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70850
--- Comment #3 from Ian Lance Taylor ---
Do we support ./configure these days?
Historically we have only supported SRCDIR/configure run in an empty directory.
||ian at airs dot com
Resolution|--- |FIXED
--- Comment #4 from Ian Lance Taylor ---
This is fixed by your patch, I think. Let me know if not.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70304
--- Comment #1 from Ian Lance Taylor ---
This looks like a disagreement between the Solaris assembler and the GNU
objcopy program. I don't know if there is anything that gccgo can do to fix
it. You should probably report this as a GNU binutils
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70182
--- Comment #11 from Ian Lance Taylor ---
g++ has generated (what I think is) the incorrect mangling since at least GCC
4.7, so this probably needs to be controlled by yet another -fabi-version
value.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70182
--- Comment #9 from Ian Lance Taylor ---
Yes. The relevant bits in the mangling ABI are
::= [gs]
::= sr
::= srN + E
::= [gs] sr + E
::=
::= on
::= on
::= dn
::=
::= [ ]
::=
srT_ must be sr ,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70182
Ian Lance Taylor changed:
What|Removed |Added
CC||ian at airs dot com
--- Comment #7
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69966
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67180
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |WAITING
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68562
Ian Lance Taylor changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68503
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66904
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66368
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |WAITING
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65134
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68255
Ian Lance Taylor changed:
What|Removed |Added
CC||e29253 at jp dot ibm.com
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63731
Ian Lance Taylor changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64001
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |WAITING
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68420
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |SUSPENDED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68562
--- Comment #4 from Ian Lance Taylor ---
Created attachment 37650
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37650=edit
Possible patch
Do you have a way to test whether this patch fixes the problem?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69511
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68562
--- Comment #6 from Ian Lance Taylor ---
Your build is passing the option "-B/usr/local/x86_64-pc-linux-gnu/lib/"; why?
You have an old version of the Go libraries installed under that directory.
Either remove the -B option or remove the old
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69357
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68980
Ian Lance Taylor changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69050
--- Comment #1 from Ian Lance Taylor ---
The algorithms aren't unrelated. unit_addrs_compare sorts by low then high
then lineoff. unit_addrs_search returns any entry that includes the PC for
which we are searching, where it includes the PC if
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68931
--- Comment #1 from Ian Lance Taylor ---
I don't understand your patch, because I don't understand where CONFIG_USE_MUSL
gets set.
It also seems like the wrong approach. The basic goal of configure tests is to
test for features, not systems.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67198
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66147
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65717
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68477
Ian Lance Taylor changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61303
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68496
--- Comment #1 from Ian Lance Taylor ---
I can not recreate this problem. It works fine for me.
The stack trace is incomplete for some reason so I don't know what is going
wrong.
If you cd into x86_64-pc-linux-gnu/libgo, you can run
make
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66574
--- Comment #7 from Ian Lance Taylor ---
I'm a little worried because I'm not 100% confident that clock_gettime is
available in the standard library on all systems. I don't want to break GCC 5
is that is the case. Admittedly I haven't seen any
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68496
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66368
--- Comment #9 from Ian Lance Taylor ---
I'm not having any luck reproducing this. I built a 32-bit PPC GNU/Linux (on
the GCC compile farm, which is a PPC64 machine, using glibc 2.18). I deleted
the libgo files and rebuilt them with
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68072
--- Comment #9 from Ian Lance Taylor ---
Thanks very much for taking the time to narrow down the test case. That is
nice and simple.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68072
--- Comment #10 from Ian Lance Taylor ---
FYI, refiled as https://golang.org/issue/13344 and sent out
https://golang.org/cl/17151 , a variant of the patch you suggested.
||2015-11-21
CC||ian at airs dot com
Ever confirmed|0 |1
--- Comment #2 from Ian Lance Taylor ---
Sent https://golang.org/cl/17154 to fix this in the master libgo sources.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68072
Ian Lance Taylor changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67976
Ian Lance Taylor changed:
What|Removed |Added
Status|RESOLVED|ASSIGNED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67976
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67976
Ian Lance Taylor changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Version|6.0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68141
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66574
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65785
Ian Lance Taylor changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66406
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66378
Ian Lance Taylor changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68383
--- Comment #5 from Ian Lance Taylor ---
Thanks for the test case. Which version of GCC are you using to compile? When
I try to compile the test case I get
/usr/lib/gcc/x86_64-pc-linux-gnu/4.9.3/include/avx512fintrin.h: In function
‘__m512i
501 - 600 of 1607 matches
Mail list logo
