[llvm-branch-commits] [lld, wasm] Replace config-> with ctx.arg. (PR #119835)
Sam Clegg via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Fri Dec 13 10:23:34 PST 2024
https://github.com/sbc100 commented:
Could you add a little context to the PR description? i.e. Why is this better?
Does this change also remove the old `config` global or just stop using it?
Objectively this still like a regression in readability since it adds more visual noise and indirection. However, I get that we are trying to make lld more library-friendly. I'm just a little sad that it comes are this readability cost :(
https://github.com/llvm/llvm-project/pull/119835
More information about the llvm-branch-commits
mailing list