[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