arichardson added a comment.

In D78478#2128604 <https://reviews.llvm.org/D78478#2128604>, @MaskRay wrote:

> The idea look good to me, but I want some opinions on the name `UTC_ARGS` (I 
> can't help associating it with Coordinated Universal Time). Adding some folks 
> who may have opinions: @greened @lebedev.ri  @RKSimon @spatel  @xbolva00


I agree that UTC_ARGS could be a bit confusing. However, there are currently 
145 UTC_ARGS uses in tests/Transforms and I'd rather not update them as part of 
this change.

I feel that changing the name should be a separate change that can be committed 
after this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78478



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

Reply via email to