[PATCH] D44094: [ELF] - Report LMA region overflows.

[George Rimar via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rLLD326892: [ELF] - Report LMA region overflows. (authored by grimar, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D44094?vs=137007&id=137360#toc

Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D44094

Files:
  ELF/LinkerScript.cpp
  test/ELF/linkerscript/lma-overflow.test


Index: ELF/LinkerScript.cpp
===================================================================
--- ELF/LinkerScript.cpp
+++ ELF/LinkerScript.cpp
@@ -117,9 +117,9 @@
   if (Ctx->MemRegion)
     expandMemoryRegion(Ctx->MemRegion, Size, Ctx->MemRegion->Name,
                        Ctx->OutSec->Name);
-  // FIXME: check LMA region overflow too.
   if (Ctx->LMARegion)
-    Ctx->LMARegion->CurPos += Size;
+    expandMemoryRegion(Ctx->LMARegion, Size, Ctx->LMARegion->Name,
+                       Ctx->OutSec->Name);
 }
 
 void LinkerScript::setDot(Expr E, const Twine &Loc, bool InSec) {
Index: test/ELF/linkerscript/lma-overflow.test
===================================================================
--- test/ELF/linkerscript/lma-overflow.test
+++ test/ELF/linkerscript/lma-overflow.test
@@ -0,0 +1,17 @@
+# REQUIRES: x86
+
+# echo '.section .foo,"a"; .quad 1' | llvm-mc -filetype=obj -triple=x86_64-unknown-linux - -o %t.o
+# RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux %t.s -o %t.o
+# RUN: not ld.lld -o %t %t.o --script %s 2>&1 | FileCheck %s
+# CHECK: error: section '.foo' will not fit in region 'flash': overflowed by 264 bytes
+
+MEMORY {
+  ram (rwx) : org = 0x1000, len = 0x300
+  flash (rwx) : org = 0x1000, len = 0x100
+}
+SECTIONS {
+  .foo : {
+    *(.foo)
+    . += 0x200;
+  } > ram AT>flash
+}


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44094.137360.patch
Type: text/x-patch
Size: 1326 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180307/9f978434/attachment.bin>