This is an archive of the discontinued LLVM Phabricator instance.

Add __builtin_canonicalize
ClosedPublic

Authored by arsenm on Feb 23 2016, 5:50 PM.

Details

Reviewers
rsmith

Diff Detail

Event Timeline

arsenm updated this revision to Diff 48868.Feb 23 2016, 5:50 PM
arsenm retitled this revision from to Add __builtin_canonicalize.
arsenm updated this object.
arsenm added a subscriber: cfe-commits.
rsmith accepted this revision.Feb 23 2016, 6:14 PM
rsmith added a reviewer: rsmith.
rsmith added a subscriber: rsmith.

Sure, it makes sense to expose this.

docs/LanguageExtensions.rst
1743

The cross-reference to the LLVM documentation here is useful, but please lead with a (brief) description of what this builtin does (the first sentence from the LLVM doc would be fine).

include/clang/Basic/Builtins.def
381–382

Why is there no long double version of this builtin?

This revision is now accepted and ready to land.Feb 23 2016, 6:14 PM
arsenm updated this revision to Diff 48879.Feb 23 2016, 7:34 PM
arsenm edited edge metadata.

Add long double version. Add the short summary before linking to the intrinsic doc

arsenm closed this revision.Feb 27 2016, 1:11 AM

r262122