[PATCH] D109634: [LLD] Remove global state in lld/COFF
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 10 15:20:43 PDT 2021
MaskRay added a comment.
LGTM.
For other contexts (e.g. LLVMContext, MCContext), the context argument is more commonly used as the first variable.
Since it is always passed, doing do can potentially allow default arguments, though default arguments are not common in lld/.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109634/new/
https://reviews.llvm.org/D109634
More information about the llvm-commits
mailing list