[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