This is an archive of the discontinued LLVM Phabricator instance.

[PowerPC] Implement longdouble pack/unpack builtins
ClosedPublic

Authored by qiucf on Oct 19 2021, 12:53 AM.

Details

Diff Detail

Event Timeline

qiucf created this revision.Oct 19 2021, 12:53 AM
qiucf requested review of this revision.Oct 19 2021, 12:53 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptOct 19 2021, 12:53 AM
jsji accepted this revision as: jsji.Oct 26 2021, 8:12 AM

LGTM with some nits.

clang/lib/Sema/SemaChecking.cpp
3382

Can you please add reference to documents that define these builtins in descriptions?

llvm/test/CodeGen/PowerPC/longdouble-pack.ll
3

Can we also include AIX/big endian run lines?

llvm/utils/TableGen/IntrinsicEmitter.cpp
253

Additional ,?

This revision is now accepted and ready to land.Oct 26 2021, 8:12 AM
This revision was landed with ongoing or failed builds.Nov 3 2021, 3:04 AM
This revision was automatically updated to reflect the committed changes.
qiucf marked 3 inline comments as done.