martong added inline comments.
================
Comment at:
clang-tools-extra/clang-tidy/bugprone/EasilySwappableParametersCheck.cpp:550-551
+ if (isUselessSugar(LType.getTypePtr())) {
+ LLVM_DEBUG(llvm::dbgs()
+ << "--- calculateMixability. LHS is useless sugar.\n");
return calculateMixability(Check, LType.getSingleStepDesugaredType(Ctx),
----------------
Is this still WIP or do you use the DEBUG printouts in the tests? If not then
could you please create a Diff without the DEBUG printouts, that could increase
readability and ease the review.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75041/new/
https://reviews.llvm.org/D75041
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
