This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Simplify getFloatTypeSemantics
ClosedPublic

Authored by MaskRay on Jul 9 2019, 8:55 AM.

Details

Summary

When the float point representations are the same on the host and on the target device,
(&Target->getLongDoubleFormat() == &AuxTarget->getLongDoubleFormat()),
we can just use AuxTarget->getLongDoubleFormat().

Diff Detail

Repository
rL LLVM

Event Timeline

MaskRay created this revision.Jul 9 2019, 8:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 9 2019, 8:55 AM

Could we generally require some message for all reviews and commits?

ABataev accepted this revision.Jul 9 2019, 9:30 AM

LG, but I agree, some general description would be good

This revision is now accepted and ready to land.Jul 9 2019, 9:30 AM
MaskRay updated this revision to Diff 208780.Jul 9 2019, 12:35 PM
MaskRay edited the summary of this revision. (Show Details)

Update description

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptJul 9 2019, 12:36 PM