[PATCH] D41397: [ELF] - Fix for ld.lld does not accept "AT" syntax for declaring LMA region

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 9 15:22:34 PST 2018


ruiu added inline comments.


================
Comment at: ELF/LinkerScript.cpp:676
   }
+  if (MemoryRegion *MR = findMemoryRegion(Sec->LMARegionName)) {
+    uint64_t Offset = MR->Origin - Dot;
----------------
Inline `findMemoryRegion` and delete that function.


================
Comment at: ELF/LinkerScript.cpp:802
         continue;
-      Sec->MemRegion = findMemoryRegion(Sec);
+      Sec->MemRegion = matchMemoryRegion(Sec);
       // Handle align (e.g. ".foo : ALIGN(16) { ... }").
----------------
Rename this `findMemoryRegion`.


https://reviews.llvm.org/D41397





More information about the llvm-commits mailing list