This is an archive of the discontinued LLVM Phabricator instance.

clang: Add __builtin_elementwise canonicalize and copysign
ClosedPublic

Authored by arsenm on Dec 8 2022, 8:46 AM.

Details

Reviewers
fhahn
scanon
Summary

Just copy paste from the other functions. I also need fma, but
the current code seems to assume 1 or 2 arguments.

Diff Detail

Event Timeline

arsenm created this revision.Dec 8 2022, 8:46 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 8 2022, 8:46 AM
arsenm requested review of this revision.Dec 8 2022, 8:46 AM
fhahn added inline comments.Dec 9 2022, 3:56 PM
clang/docs/LanguageExtensions.rst
644

I think this needs re-flowing , otherwise the table won't be rendered properly? Same for the next line

646

T -> x? Also, the argument y is lowercase, so Y -> y?

arsenm updated this revision to Diff 481812.Dec 9 2022, 6:45 PM

Try to fix formatting

arsenm marked an inline comment as done.Dec 9 2022, 6:45 PM
arsenm added inline comments.
clang/docs/LanguageExtensions.rst
644

I'm not really sure what's going on with the formatting or how this works. It looks worse on Phabricator than in the editor

fhahn accepted this revision.Dec 13 2022, 3:31 PM

LGTM, but please make sure the .rst file builds with sphinx.

clang/docs/LanguageExtensions.rst
644

I just tried this locally and the entries seem to be mis-aligned, which gets rejected by sphinx:

Warning, treated as error:
../tools/clang/docs/LanguageExtensions.rst:644:Malformed table.
Text in column margin in table line 16.
This revision is now accepted and ready to land.Dec 13 2022, 3:31 PM
arsenm closed this revision.Dec 21 2022, 3:02 PM
clang/docs/LanguageExtensions.rst
644

It builds for me (although the line wrapping for these and the others are awkward, splitting the parameter names). Probably need to specifically widen the name column