[PATCH] D142274: [ADT] Add llvm::byteswap to bit.h

Kazu Hirata via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 21 08:18:21 PST 2023


kazu marked an inline comment as done.
kazu added a comment.

Please take a look.  Thanks!



================
Comment at: llvm/unittests/ADT/BitTest.cpp:126
+            llvm::byteswap(int64_t(0x8877665544332211LL)));
+}
+
----------------
RKSimon wrote:
> Why not just merge these into a single TEST(BitTest, ByteSwap) ?
I've grouped small ones.  I've left the ones involving loops untouched as they are a little big.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142274



More information about the llvm-commits mailing list