[PATCH] D30419: [ELF] - Define __bss_start symbol.

Rafael Ávila de Espíndola via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 30 06:20:44 PDT 2017


rafael added a comment.

Rui, can you check with whatever is using this what the intended behavior would be when there is a .bss and a .bss.rel.ro? I would guess it wants to see .bss since .bss.rel.ro is ro at runtime.



================
Comment at: ELF/Writer.cpp:849
 
+  // __bss_start is the location of .bss.* sections.
+  ElfSym::BssStart =
----------------
just say ".bss". It is the name of the output section.



https://reviews.llvm.org/D30419





More information about the llvm-commits mailing list