On 2017-09-19 07:47 AM, Matt Madison wrote:
On Mon, Sep 18, 2017 at 4:26 PM, Khem Raj <raj.k...@gmail.com> wrote:
On Mon, Sep 18, 2017 at 3:52 PM, Mark Hatle <mark.ha...@windriver.com> wrote:
On 9/18/17 5:50 PM, Matt Madison wrote:

On 18 Sep 2017, at 2:11 PM, Slater, Joseph <joe.sla...@windriver.com
<mailto:joe.sla...@windriver.com>> wrote:

This is the default tune for qemumips.  It appears that the problem does not
occur for an oe-core checkout just before the split creating go-runtime.

Before the split, go-helloworld will build, but after the split it will not
because it needs go-runtime.

There’s a TUNE_CCARGS_remove in go_1.9.bb to remove the ‘-march=mips32r2’ flag
that probably either needs to be moved to a .inc file or copied to
go-runtime_1.9.bb.  Missed that when splitting out the runtime recipe.

That seems wrong.  We should be using the TUNE_CCARGS, and -not- using the
arguments that 'go' has decided must be right.

Otherwise the optimizations may or may not be compatible... and if someone
selects different arguments, they won't get filtered.

go supports mips32r1, and needs fpu or kernel FPU emulation enabled
so using mips32r2 might be fine back when I was adding it to oe-core it
did not work, things might be different with 1.9 release where mips
support is complete in golang.

It's still mips32r1, though, and the go build tool explicitly adds
'-march=mips32 -mabi=32' when invoking the C compiler for cgo.

-Matt


Is someone working on a fix or should I open a YP defect?

--
# Randy MacLeod.  WR Linux
# Wind River an Intel Company
--
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to