[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