cjdb added a comment. > That said, I think this particular check is more mature and closer to landing > than the other one. Based on the amount of effort in this patch already, > unless there are strong objections, I think we should base the "make this > const if it can be made so" checks on this one.
Ack. How about we worry about D107873 <https://reviews.llvm.org/D107873> once this patch lands, and once I've had a chance to rebase it on top of this? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54943/new/ https://reviews.llvm.org/D54943 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits