New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
x/arch/arm64/arm64asm: tests fail with new binutils #27754
Comments
I think what's going on here is that the new binutils in this version of Ubuntu supports some new mnemonics. I presume the fix is to adjust allowedMismatchObjdump to accept both forms, I'll submit a CL to implement this soon but I'm a bit busy today. |
Yeah, I think this is the cause. Is there some flags to objdump to make it prefer old instruction names? If so, we probably want to use that so the test is more tolerable to new objdump changes. |
You can pass -M no-aliases to, I think, suppress all mnemonics but that's probably going too far. |
Change https://golang.org/cl/136455 mentions this issue: |
Fixes golang/go#27754 Change-Id: I8fcc3bc3c718cf0d93afbd1d383df48316b522d4 Reviewed-on: https://go-review.googlesource.com/136455 Run-TryBot: Michael Hudson-Doyle <michael.hudson@canonical.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Cherry Zhang <cherryyz@google.com>
Fixes golang/go#27754 Change-Id: I8fcc3bc3c718cf0d93afbd1d383df48316b522d4 Reviewed-on: https://go-review.googlesource.com/136455 Run-TryBot: Michael Hudson-Doyle <michael.hudson@canonical.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Cherry Zhang <cherryyz@google.com>
Fixes golang/go#27754 Change-Id: I8fcc3bc3c718cf0d93afbd1d383df48316b522d4 Reviewed-on: https://go-review.googlesource.com/136455 Run-TryBot: Michael Hudson-Doyle <michael.hudson@canonical.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Cherry Zhang <cherryyz@google.com>
Fixes golang/go#27754 Change-Id: I8fcc3bc3c718cf0d93afbd1d383df48316b522d4 Reviewed-on: https://go-review.googlesource.com/136455 Run-TryBot: Michael Hudson-Doyle <michael.hudson@canonical.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Cherry Zhang <cherryyz@google.com>
Fixes golang/go#27754 Change-Id: I8fcc3bc3c718cf0d93afbd1d383df48316b522d4 Reviewed-on: https://go-review.googlesource.com/136455 Run-TryBot: Michael Hudson-Doyle <michael.hudson@canonical.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Cherry Zhang <cherryyz@google.com>
What version of Go are you using (
go version
)?1.10 but it doesn't matter.
Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (
go env
)?Ubuntu 18.10 on arm64.
What did you do?
What did you expect to see?
tests passing
What did you see instead?
The text was updated successfully, but these errors were encountered: