[PATCH] CodeGen: implement __emit intrinsic

Reid Kleckner rnk at google.com
Mon Dec 15 18:20:19 PST 2014


Do users assume that physical registers are live from one __emit blob to the next?  If so, I think we need to treat this as proper inline assembly and merge the blobs together like we would for __asm. Otherwise, LLVM makes no guarantees that it won't insert spills between the blobs.


================
Comment at: test/CodeGen/builtins-arm-msvc-compat-error.c:8
@@ +7,3 @@
+
+// CHECK: fatal error: error in backend: constant value required!
+
----------------
Why isn't sema catching this? I thought we had a builtin character code for that.

http://reviews.llvm.org/D3489

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list