[PATCH] D43642: [ELF] Add comment for preemptible and fix typo. NFC

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 22 18:10:57 PST 2018


ruiu added inline comments.


================
Comment at: ELF/Symbols.h:157
 
+  // True if this symbol is preemptible at link or load time.
   unsigned IsPreemptible : 1;
----------------
Is this correct? I thought IsPreemptible=true means the symbol is preemptible at load time. I don't think "a symbol preemptible at link time" is a well defined concept.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D43642





More information about the llvm-commits mailing list