diff options
| author | Andrew Wilkins <axwalk@gmail.com> | 2015-09-25 06:28:14 +0000 |
|---|---|---|
| committer | Andrew Wilkins <axwalk@gmail.com> | 2015-09-25 06:28:14 +0000 |
| commit | f69d45efc3c119c53cd725b422d88fe7edcfdb10 (patch) | |
| tree | a39424daccd867bc7b88e98cf2b49e8fe58bfe72 /llgo/irgen/compiler.go | |
| parent | 8016f3a4f5d5fd009a5c91d7d66eb6db3942daac (diff) | |
| download | bcm5719-llvm-f69d45efc3c119c53cd725b422d88fe7edcfdb10.tar.gz bcm5719-llvm-f69d45efc3c119c53cd725b422d88fe7edcfdb10.zip | |
[llgo] irgen: always use TargetMachine's data layout
Summary:
Another attempt at resolving the runtime assertion
in llgoi due to data layout mismatch between module
and execution engine.
The X86 data layout constant appears to be unnecessary,
and does not match what the execution engine picks.
Using the registered Target, we pick the same data
layout as the execution engine.
While I was in the vicinity, I deleted the last
remnants of PNaCl support.
Reviewers: pcc
Subscribers: jfb, llvm-commits, dschuff
Differential Revision: http://reviews.llvm.org/D12630
llvm-svn: 248565
Diffstat (limited to 'llgo/irgen/compiler.go')
| -rw-r--r-- | llgo/irgen/compiler.go | 13 |
1 files changed, 0 insertions, 13 deletions
diff --git a/llgo/irgen/compiler.go b/llgo/irgen/compiler.go index 8f4e65f1aa6..c00a881a42a 100644 --- a/llgo/irgen/compiler.go +++ b/llgo/irgen/compiler.go @@ -20,7 +20,6 @@ import ( "log" "sort" "strconv" - "strings" llgobuild "llvm.org/llgo/build" "llvm.org/llgo/debug" @@ -110,15 +109,10 @@ type CompilerOptions struct { type Compiler struct { opts CompilerOptions dataLayout string - pnacl bool } func NewCompiler(opts CompilerOptions) (*Compiler, error) { compiler := &Compiler{opts: opts} - if strings.ToLower(compiler.opts.TargetTriple) == "pnacl" { - compiler.opts.TargetTriple = PNaClTriple - compiler.pnacl = true - } dataLayout, err := llvmDataLayout(compiler.opts.TargetTriple) if err != nil { return nil, err @@ -133,7 +127,6 @@ func (c *Compiler) Compile(fset *token.FileSet, astFiles []*ast.File, importpath CompilerOptions: c.opts, dataLayout: c.dataLayout, target: target, - pnacl: c.pnacl, llvmtypes: NewLLVMTypeMap(llvm.GlobalContext(), target), } return compiler.compile(fset, astFiles, importpath) @@ -151,12 +144,6 @@ type compiler struct { llvmtypes *llvmTypeMap types *TypeMap - // pnacl is set to true if the target triple was originally - // specified as "pnacl". This is necessary, as the TargetTriple - // field will have been updated to the true triple used to - // compile PNaCl modules. - pnacl bool - debug *debug.DIBuilder } |

