This is an archive of the discontinued LLVM Phabricator instance.

[SimplifyCFG] use fshr instead of shl/lshr/or
AbandonedPublic

Authored by shawnl on Apr 25 2019, 2:02 PM.

Details

Reviewers
jmolloy
Summary

We already try (but fail due to lack of sub op)
to convert this to fshr in AggressiveInstCombine.cpp:92.

If a target cannot lower fshr/fshl then it needs to be fixed.
Performance on targets that have such an instruction should
not be penalized.

Diff Detail

Repository
rL LLVM

Event Timeline

shawnl created this revision.Apr 25 2019, 2:02 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 25 2019, 2:02 PM
shawnl abandoned this revision.Apr 25 2019, 4:17 PM

somehow ended up with two of these