[PATCH] D30724: Dont emit Mapping symbols for Non allocatable sections.

Tim Northover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 8 08:02:59 PST 2017


t.p.northover added inline comments.


================
Comment at: lib/Target/ARM/MCTargetDesc/ARMELFStreamer.cpp:479
+    MCSectionELF *CurSection = cast<MCSectionELF>(Section);
+    if (CurSection->UseCodeAlign())
+      LastEMS = EMS_None;
----------------
Now this is looking a lot more promising! But I don't see the need for any exceptions to an EMS_Data default. That seems to be exactly what the ABI says should happen.


https://reviews.llvm.org/D30724





More information about the llvm-commits mailing list