[PATCH] Fix PR18345: ldr= pseudo instruction produces incorrect code when using in inline assembly

Saleem Abdulrasool compnerd at compnerd.org
Mon Jan 27 22:06:37 PST 2014



================
Comment at: lib/Target/ARM/MCTargetDesc/ARMELFStreamer.cpp:108
@@ -107,3 @@
-void ARMTargetStreamer::anchor() {}
-ARMTargetStreamer::ARMTargetStreamer(MCStreamer &S) : MCTargetStreamer(S) {}
-
----------------
I dont see this being moved to the header, am I misunderstanding something or is there now a missing constructor?

================
Comment at: lib/Target/ARM/MCTargetDesc/ARMTargetStreamer.cpp:177
@@ +176,3 @@
+//
+ARMTargetMachOStreamer::ARMTargetMachOStreamer(MCStreamer &S)
+    : ARMTargetStreamer(S) {}
----------------
Would you mind splitting out ARMTargetMachOStreamer into a separate file?


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



More information about the llvm-commits mailing list