WebAssembly: remove 'external' from test
authorJF Bastien <jfb@google.com>
Wed, 23 Dec 2015 23:56:13 +0000 (23:56 +0000)
committerJF Bastien <jfb@google.com>
Wed, 23 Dec 2015 23:56:13 +0000 (23:56 +0000)
commitc2a378514142a08dead206a95a967095a4e0d7eb
tree5433d6d5889e585e2e5db546a172a5b6e94a50f5
parentda801219ba8d6d2a8663d4dd3c14e8e3fca35ba5
WebAssembly: remove 'external' from test

Summary: Linker testing was sad at seeing an unresolved external symbol. For now don't do that: it's valid but we're not playing with multi-file linking yet, and the LLVM tests are used as hacky sanity tests for single-file linking (the GCC torture tests are much better for this purpose). Another solution would be to use '.extern' to make the intent explicit (don't simple-file link this, there's an unresolved symbol), some assemblers use '.extern' while others ignore it, so we wouldn't really be inventing anything new.

Reviewers: sunfish, kripken

Subscribers: jfb, llvm-commits, dschuff

Differential Revision: http://reviews.llvm.org/D15753

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@256353 91177308-0d34-0410-b5e6-96231b3b80d8
test/CodeGen/WebAssembly/offset-folding.ll