[lld] [NFC][ELF] Replace DynamicReloc::Kind with the equivalent bool in APIs (PR #150813)

Peter Collingbourne via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 1 10:11:33 PDT 2025


pcc wrote:

> Thanks for cleaning up my ugly https://reviews.llvm.org/D100490 in the previous commits :)
> 
> I think having names for the boolean parameters makes the code calling this function easier to read than a magic true/false. So I'd have a slight preference for keeping an enum parameter but I'll leave that decision to @MaskRay.

+1, the enum parameters were more readable IMHO.

https://github.com/llvm/llvm-project/pull/150813


More information about the llvm-commits mailing list