Page MenuHomePhabricator

[FPEnv] EarlyCSE support for constrained intrinsics, default FP environment edition
Needs ReviewPublic

Authored by kpn on Apr 6 2021, 8:21 AM.

Details

Summary

EarlyCSE cannot distinguish between floating point instructions and constrained floating point intrinsics that are marked as running in the default FP environment. Said intrinsics are supposed to behave exactly the same as the regular FP instructions. Teach EarlyCSE to handle them in that case.

Diff Detail

Event Timeline

kpn created this revision.Apr 6 2021, 8:21 AM
kpn requested review of this revision.Apr 6 2021, 8:21 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 6 2021, 8:21 AM

early returns?

llvm/lib/IR/IntrinsicInst.cpp
201

return false;

207

return false;

kpn updated this revision to Diff 335647.Apr 6 2021, 1:25 PM

Address review comments: use early return.