[PATCH] ARM IAS: support .movsp
Logan Chien
tzuhsiang.chien at gmail.com
Mon Jan 20 07:20:17 PST 2014
================
Comment at: lib/Target/ARM/MCTargetDesc/ARMELFStreamer.cpp:170
@@ +169,3 @@
+ "the operand of .movsp cannot be either sp or pc");
+ assert(UsedFP && ".setfp must precede .movsp");
+
----------------
This line will cause compilation error.
================
Comment at: lib/Target/ARM/MCTargetDesc/ARMELFStreamer.cpp:169
@@ +168,3 @@
+ assert((Reg != ARM::SP && Reg != ARM::PC) &&
+ "the operand of .movsp cannot be either sp or pc");
+ assert(UsedFP && ".setfp must precede .movsp");
----------------
It seems that you have added these assertions to address Renato's comments. However, I feel this is not necessary. IMO, the assembler which will assemble the generated assembly will perform such check. How do you think, Renato?
http://llvm-reviews.chandlerc.com/D2530
More information about the llvm-commits
mailing list