HazardyKnusperkeks accepted this revision.
HazardyKnusperkeks added a comment.

In D141654#4050852 <https://reviews.llvm.org/D141654#4050852>, @rymiel wrote:

> (This needs to run the updated dump script from D138446 
> <https://reviews.llvm.org/D138446>)
>
> I'm not sure what the strict //benefit// of squishing the two options 
> together is, but I support it

Having just one option instead of two, for one feature is better, especially 
when the two options are that far apart in the documentation.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D141654/new/

https://reviews.llvm.org/D141654

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to