Re: [PATCH] D16993: Add missing __builtin_bitreverse8

2016-03-23 Thread Matt Arsenault via cfe-commits
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

Re: [PATCH] D16993: Add missing __builtin_bitreverse8

2016-03-23 Thread Matt Arsenault via 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

Re: [PATCH] D16993: Add missing __builtin_bitreverse8

2016-03-23 Thread Richard Smith via cfe-commits
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

Re: [PATCH] D16993: Add missing __builtin_bitreverse8

2016-03-21 Thread Matt Arsenault via cfe-commits
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

Re: [PATCH] D16993: Add missing __builtin_bitreverse8

2016-03-07 Thread Matt Arsenault via 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:

Re: [PATCH] D16993: Add missing __builtin_bitreverse8

2016-03-07 Thread Matt Arsenault via cfe-commits
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