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

Reply via email to