[PATCH] D12623: [llgo] clear data layout before adding module to JIT

[Andrew Wilkins via llvm-commits]

axw abandoned this revision.

================
Comment at: cmd/llgoi/llgoi.go:155
@@ +154,3 @@
+	// Clear the data layout; the JIT wants to set it.
+	module.Module.SetDataLayout("")
+
----------------
joker.eph wrote:
> There i something fishy, why is the datalayout set to something else than the target in the first place?
http://llvm.org/klaus/llgo/blob/master/irgen/targets.go#L-43

I don't recall how that came to be, but most likely that needs to be fixed. I'll close this and look into that, thanks.


http://reviews.llvm.org/D12623