[PATCH] D139640: clang: Add __builtin_elementwise canonicalize and copysign

2022-12-21 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm closed this revision. arsenm added a comment. 437346abe18ec4fc982ae36f6821487dafc1a06e Comment at: clang/docs/LanguageExtensions.rst:644 magnitude than x

[PATCH] D139640: clang: Add __builtin_elementwise canonicalize and copysign

2022-12-13 Thread Florian Hahn via Phabricator via cfe-commits
fhahn accepted this revision. fhahn added a comment. This revision is now accepted and ready to land. LGTM, but please make sure the `.rst` file builds with `sphinx`. Comment at: clang/docs/LanguageExtensions.rst:644 magnitude than

[PATCH] D139640: clang: Add __builtin_elementwise canonicalize and copysign

2022-12-09 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm marked an inline comment as done. arsenm added inline comments. Comment at: clang/docs/LanguageExtensions.rst:644 magnitude than x + T__builtin_elementwise_canonicalize(T x) return the platform specific canonical encoding of

[PATCH] D139640: clang: Add __builtin_elementwise canonicalize and copysign

2022-12-09 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm updated this revision to Diff 481812. arsenm added a comment. Try to fix formatting CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139640/new/ https://reviews.llvm.org/D139640 Files: clang/docs/LanguageExtensions.rst clang/include/clang/Basic/Builtins.def

[PATCH] D139640: clang: Add __builtin_elementwise canonicalize and copysign

2022-12-09 Thread Florian Hahn via Phabricator via cfe-commits
fhahn added inline comments. Comment at: clang/docs/LanguageExtensions.rst:644 magnitude than x + T__builtin_elementwise_canonicalize(T x) return the platform specific canonical encoding of a floating-point number

[PATCH] D139640: clang: Add __builtin_elementwise canonicalize and copysign

2022-12-08 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm created this revision. arsenm added reviewers: fhahn, scanon. Herald added a project: All. arsenm requested review of this revision. Herald added a subscriber: wdng. Just copy paste from the other functions. I also need fma, but the current code seems to assume 1 or 2 arguments.