Clean up the handling of inline asm.

Make inline asm a report_fatal_error instead of an assertion failure so
that it's a little friendlier, and add a test to make sure llc -march=js
rejects inline asm.

Also disable the PNaCl inline asm("":::"memory") lowering pass. If people
are using this, it's best that we diagnose it as it likely isn't portable.
There are usually better alternatives.

1 files changed, 13 insertions, 0 deletions

diff --git a/test/CodeGen/JS/asm.ll b/test/CodeGen/JS/asm.ll
new file mode 100644
index 0000000000..41a30431da
--- /dev/null
+++ b/test/CodeGen/JS/asm.ll
@@ -0,0 +1,13 @@
+; RUN: not llc -march=js < %s
+
+; Inline asm isn't supported (yet?). llc should report an error when it
+; encounters inline asm.
+;
+; We could support the special case of an empty inline asm string without much
+; work, but code that uses such things most likely isn't portable anyway, and
+; there are usually much better alternatives.
+
+define void @foo() {
+  call void asm "", ""()
+  ret void
+}