[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