This is an archive of the discontinued LLVM Phabricator instance.

ConstantFolding: Handle exp10 intrinsic
ClosedPublic

Authored by arsenm on Aug 14 2023, 8:45 AM.

Diff Detail

Event Timeline

arsenm created this revision.Aug 14 2023, 8:45 AM
arsenm requested review of this revision.Aug 14 2023, 8:45 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 14 2023, 8:45 AM
Herald added a subscriber: wdng. · View Herald Transcript
kpn added a comment.Aug 14 2023, 11:46 AM

There are a lot more places in ConstantFolding.cpp where exp2 appears. Don't those need exp10 support alongside exp2 as well?

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

kpn accepted this revision.Aug 14 2023, 12:03 PM

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.

This revision is now accepted and ready to land.Aug 14 2023, 12:03 PM