[PATCH] D50616: [Fixed Point Arithmetic] FixedPointCast

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 20 12:29:55 PDT 2018


rjmccall added a comment.

In https://reviews.llvm.org/D50616#1206181, @leonardchan wrote:

> I made a post on llvm-dev (http://lists.llvm.org/pipermail/llvm-dev/2018-August/125433.html) to see if other people have opinions on this. In the meantime, do you think I should make a separate patch that moves this into an LLVM intrinsic function?


Yeah, I think that even if LLVM decides they don't want to include first-class IR support for fixed-point types, it makes more sense to provide standard intrinsics for these operations than to do all of the lowering in the frontend.


Repository:
  rC Clang

https://reviews.llvm.org/D50616





More information about the cfe-commits mailing list