This is an archive of the discontinued LLVM Phabricator instance.

[clang][Interp] __builtin_copysign
ClosedPublic

Authored by tbaeder on Jul 15 2023, 6:28 AM.

Diff Detail

Event Timeline

tbaeder created this revision.Jul 15 2023, 6:28 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 15 2023, 6:28 AM
tbaeder requested review of this revision.Jul 15 2023, 6:28 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 15 2023, 6:28 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript

Missing test coverage for copysign.

clang/lib/AST/Interp/InterpBuiltin.cpp
146

Should split out the fmin functionality into a separate patch.

I can add some basic tests for __builtin_copysign here, but there are test added in https://reviews.llvm.org/D155369 as well.

tbaeder updated this revision to Diff 541325.Jul 17 2023, 11:01 PM
tbaeder marked an inline comment as done.
This revision is now accepted and ready to land.Jul 18 2023, 7:20 AM
This revision was landed with ongoing or failed builds.Jul 28 2023, 1:55 AM
This revision was automatically updated to reflect the committed changes.