sammccall added a comment.

In D126717#3548184 <https://reviews.llvm.org/D126717#3548184>, @mstorsjo wrote:

> Looks reasonable to me. The variable name feels a bit un-namespaced though, 
> but that's what the tool is called currently. Should we rename the tool and 
> the variable to give it a bit more of a namespace prefix, like 
> clang-pseudo-gen or so?

That makes sense. I'll do that first and rename to CLANG_PSEUDO_GEN.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126717

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

Reply via email to