Re: [PATCH] D16482: Add builtins for bitreverse intrinsic

2016-02-03 Thread Hal Finkel via cfe-commits
hfinkel added a comment. In http://reviews.llvm.org/D16482#343177, @arsenm wrote: > r259671. > > Should I put some documentation for these somewhere? > docs/LanguageExtensions.rst seems to be the place, but it only has a random > subset of the current builtins. My impression is that we priori

Re: [PATCH] D16482: Add builtins for bitreverse intrinsic

2016-02-03 Thread Matt Arsenault via cfe-commits
arsenm closed this revision. arsenm added a comment. r259671. Should I put some documentation for these somewhere? docs/LanguageExtensions.rst seems to be the place, but it only has a random subset of the current builtins. http://reviews.llvm.org/D16482

Re: [PATCH] D16482: Add builtins for bitreverse intrinsic

2016-02-03 Thread Hal Finkel via cfe-commits
hfinkel added a subscriber: hfinkel. hfinkel accepted this revision. hfinkel added a reviewer: hfinkel. hfinkel added a comment. This revision is now accepted and ready to land. LGTM. http://reviews.llvm.org/D16482 ___ cfe-commits mailing list cfe-c