[PATCH] D46249: [LLD][ELF][AArch64] Add missing LO12 relocation to usesOnlyLowPageBits

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 3 06:34:21 PDT 2018


grimar added a comment.

In https://reviews.llvm.org/D46249#1086321, @peter.smith wrote:

> Perhaps it is worth a comment instead as the function name implies a complete list. For example "// not a complete list, excludes relocations that can always be resolved statically such as TLS local exec."


Yeah, maybe. I guess it already contains entries it never uses, btw?


https://reviews.llvm.org/D46249





More information about the llvm-commits mailing list