Skip to content

Commit f435f55

Browse files
[Lex] Use llvm::byteswap instead of sys::getSwappedBytes (NFC)
1 parent 10dff21 commit f435f55

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

clang/lib/Lex/HeaderMap.cpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -87,9 +87,8 @@ bool HeaderMapImpl::checkHeader(const llvm::MemoryBuffer &File,
8787

8888
// Check the number of buckets. It should be a power of two, and there
8989
// should be enough space in the file for all of them.
90-
uint32_t NumBuckets = NeedsByteSwap
91-
? llvm::sys::getSwappedBytes(Header->NumBuckets)
92-
: Header->NumBuckets;
90+
uint32_t NumBuckets =
91+
NeedsByteSwap ? llvm::byteswap(Header->NumBuckets) : Header->NumBuckets;
9392
if (!llvm::isPowerOf2_32(NumBuckets))
9493
return false;
9594
if (File.getBufferSize() <

clang/unittests/Lex/HeaderMapTestUtils.h

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,10 @@ template <unsigned NumBuckets, unsigned NumBytes> struct HMapFileMock {
3333
}
3434

3535
void swapBytes() {
36-
using llvm::sys::getSwappedBytes;
37-
Header.Magic = getSwappedBytes(Header.Magic);
38-
Header.Version = getSwappedBytes(Header.Version);
39-
Header.NumBuckets = getSwappedBytes(Header.NumBuckets);
40-
Header.StringsOffset = getSwappedBytes(Header.StringsOffset);
36+
Header.Magic = llvm::byteswap(Header.Magic);
37+
Header.Version = llvm::byteswap(Header.Version);
38+
Header.NumBuckets = llvm::byteswap(Header.NumBuckets);
39+
Header.StringsOffset = llvm::byteswap(Header.StringsOffset);
4140
}
4241

4342
std::unique_ptr<llvm::MemoryBuffer> getBuffer() {

0 commit comments

Comments
 (0)