[PATCH] D157892: ConstantFolding: Handle exp10 intrinsic
Kevin P. Neal via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 14 12:04:00 PDT 2023
kpn accepted this revision.
kpn added a comment.
This revision is now accepted and ready to land.
In D157892#4586059 <https://reviews.llvm.org/D157892#4586059>, @arsenm wrote:
> In D157892#4586031 <https://reviews.llvm.org/D157892#4586031>, @kpn wrote:
>
>> There are a lot more places in ConstantFolding.cpp where exp2 appears. Don't those need exp10 support alongside exp2 as well?
>
> Those are for the libcalls, not the intrinsics. They're related but not that related. I looked into handling those as a separate change but ran into https://github.com/llvm/llvm-project/issues/64677
Fair.
LGTM.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157892/new/
https://reviews.llvm.org/D157892
More information about the llvm-commits
mailing list