arsenm closed this revision.
arsenm added a comment.
r264203
http://reviews.llvm.org/D16993
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
arsenm added inline comments.
Comment at: docs/LanguageExtensions.rst:1533
@@ +1532,3 @@
+the bitpattern of an integer value; for example ``0b1234567`` becomes
+``0b7654321``.
+
rsmith wrote:
> This example doesn't make much sense: it looks like it's bit reversing
rsmith added a subscriber: rsmith.
rsmith accepted this revision.
rsmith added a reviewer: rsmith.
This revision is now accepted and ready to land.
Comment at: docs/LanguageExtensions.rst:1533
@@ +1532,3 @@
+the bitpattern of an integer value; for example ``0b1234567`` becomes
+``
arsenm added a comment.
ping
http://reviews.llvm.org/D16993
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
arsenm added a comment.
I directly copied this from the bit reverse intrinsic documentation, so that
will need to be fixed too
From: Arthur O'Dwyer
Sent: Monday, March 7, 2016 11:56 AM
To:
reviews+http://reviews.llvm.org/d16993+public+3e6e84ed28c57...@reviews.
arsenm retitled this revision from "Add documentation for bitreverse builtins"
to "Add missing __builtin_bitreverse8".
arsenm updated the summary for this revision.
arsenm updated this revision to Diff 49960.
http://reviews.llvm.org/D16993
Files:
docs/LanguageExtensions.rst
include/clang/Bas
