[PATCH] D33647: [ELF] - Linkerscript: implement NOLOAD section type.

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added inline comments.


================
Comment at: ELF/LinkerScript.h:138
   std::string MemoryRegionName;
+  bool ForceToNoBits = false;
 
----------------
`Noload` is probably a better name.


================
Comment at: ELF/ScriptParser.cpp:575
 
+  // Read section type. Only NOLOAD is supported currently.
+  // https://sourceware.org/binutils/docs/ld/Output-Section-Type.html
----------------
Read a section type. Currently, only NOLOAD is supported.


================
Comment at: ELF/ScriptParser.cpp:577
+  // https://sourceware.org/binutils/docs/ld/Output-Section-Type.html
+  if (peek() != ":") {
+    expect("(");
----------------
You can move this above `if (peek() != ":")`, no?


https://reviews.llvm.org/D33647