[PATCH] D100051: [clang] Move int <-> float scalar conversion to a separate function

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 7 11:31:04 PDT 2021


erichkeane added a comment.

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?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100051



More information about the cfe-commits mailing list