[PATCH] D50494: [LLD] Reserve memory in implicitly set LMARegions

[Konstantin Schwarz via Phabricator via llvm-commits]

kschwarz added inline comments.


================
Comment at: ELF/LinkerScript.cpp:780
+
+void LinkerScript::setLMARegion(OutputSection *Sec) {
+
----------------
grimar wrote:
> I tried to simplify the conditions in this method and ended up with the following version: D50602.
> What do you think?
Your version is also fine for me, and has better reusability. I added minor comments in the separate review.



Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D50494