[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