[PATCH] ARM: For thumb fixups store halfwords high first and low second

Saleem Abdulrasool compnerd at compnerd.org
Thu Apr 24 07:36:16 PDT 2014


Thanks for addressing all the feedback!

================
Comment at: lib/Target/ARM/MCTargetDesc/ARMAsmBackend.cpp:311
@@ +310,3 @@
+  if (IsLittleEndian) {
+    // Note that the halfwords are stored high first and low second in thumb;
+    // so we need to swap the fixup value here to map properly.
----------------
I wouldve put this as a function comment.

http://reviews.llvm.org/D3380






More information about the llvm-commits mailing list