owenpan added a comment.

In D141654#4052510 <https://reviews.llvm.org/D141654#4052510>, 
@HazardyKnusperkeks wrote:

> 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.

+1.


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