[PATCH] D84054: [lld][ELF] Add LOG2CEIL builtin ldscript function

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 23 13:17:32 PDT 2020


MaskRay added inline comments.


================
Comment at: lld/ELF/ScriptParser.cpp:1339
+      return std::ceil(
+          std::log2(std::max(a().getValue(), static_cast<uint64_t>(1))));
+    };
----------------
Use `llvm::Log2_64_Ceil` (llvm/Support/mathExtras.h)

Use `// `


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84054/new/

https://reviews.llvm.org/D84054





More information about the llvm-commits mailing list