SaurabhJha added a comment.

In D100051#2674674 <https://reviews.llvm.org/D100051#2674674>, @erichkeane 
wrote:

> Hrmph... Phab ate my other comment, which was that the name 
> `EmitCastBetweenScalarTypes` feels clunky.  Does `EmitScalarCast` or 
> `EmitScalarScalarCast` sound better and capture the meaning correctly?

Yeah, good idea. I'll rename it to `EmitScalarCast`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100051

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

Reply via email to