[PATCH] D74971: Add a llvm::shuffle and use it in lld

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 21 16:48:23 PST 2020


MaskRay accepted this revision.
MaskRay added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/include/llvm/ADT/STLExtras.h:1017
+    // but that would be stdlib dependent.
+    auto i = g() % size;
+    std::swap(*first, *(first + i));
----------------
`i` can be inlined, too.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74971/new/

https://reviews.llvm.org/D74971





More information about the llvm-commits mailing list