[PATCH] Fix inline assembly that switches between ARM and Thumb modes

Renato Golin renato.golin at linaro.org
Tue Jan 21 04:16:32 PST 2014


  Hi Greg,

  Thanks for refining this, it definitely looks better (Thanks David!). Apart from my comment on the test, LGTM.

  cheers,
  --renato


================
Comment at: test/CodeGen/ARM/inlineasm-switch-mode.ll:16
@@ +15,3 @@
+; ARM: $t
+; ARM-NEXT: 48 1c
+
----------------
adding the instruction here will make it more clear, like others


http://llvm-reviews.chandlerc.com/D2255



More information about the llvm-commits mailing list