[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