StrangeRanger added a comment. In D148467#4277960 <https://reviews.llvm.org/D148467#4277960>, @MyDeveloperDay wrote:
> I'm trying to decide if I put that option in this change or in a separate > change (thoughts @HazardyKnusperkeks, @owenpan ) I'm not too privy to your workflow, but if by "in a separate change" you mean like "in a different PR", I'd probably say implement it in a different change, for the sake of revision history. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D148467/new/ https://reviews.llvm.org/D148467 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits