This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGa82170fa41ca: [Headers] Add some intrinsic function
descriptions to immintrin.h. (authored by probinson).
Herald added a project: clang.
Changed pri
pengfei accepted this revision.
pengfei added a comment.
This revision is now accepted and ready to land.
LGTM, thanks for working on it!
Comment at: clang/lib/Headers/immintrin.h:294
+/// \param __p
+///Pointer to a 16-bit location to place the random value.
+/// \returns
probinson added a comment.
FTR, I'll be working my way through a bunch of intrinsics over the next month
or so, trying not to do too many at once. Mostly AVX2 but also some others.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D147461/new/
https://reviews.llvm.org/D147461
_
probinson created this revision.
probinson added reviewers: RKSimon, pengfei.
Herald added a project: All.
probinson requested review of this revision.
https://reviews.llvm.org/D147461
Files:
clang/lib/Headers/immintrin.h
Index: clang/lib/Headers/immintrin.h
===