[Lldb-commits] [lldb] 9984cfc - [lldb] Use llvm::byteswap in DumpRegisterValue
David Spickett via lldb-commits
lldb-commits at lists.llvm.org
Tue Apr 18 01:19:15 PDT 2023
Author: David Spickett
Date: 2023-04-18T08:18:59Z
New Revision: 9984cfc86ed6d5c1558d8dd82d82448e50d704ad
URL: https://github.com/llvm/llvm-project/commit/9984cfc86ed6d5c1558d8dd82d82448e50d704ad
DIFF: https://github.com/llvm/llvm-project/commit/9984cfc86ed6d5c1558d8dd82d82448e50d704ad.diff
LOG: [lldb] Use llvm::byteswap in DumpRegisterValue
7978abd5aef1ba84d7a1cefbc3443245acff2c48 fixed a build issue
on MSVC with some code I previously added, by adding some
ifdefs.
Now I realise that I should have been using llvm::byteswap
in the first place, which does exactly that.
Added:
Modified:
lldb/source/Core/DumpRegisterValue.cpp
Removed:
################################################################################
diff --git a/lldb/source/Core/DumpRegisterValue.cpp b/lldb/source/Core/DumpRegisterValue.cpp
index 3e59fb13b8ca..463aa5926772 100644
--- a/lldb/source/Core/DumpRegisterValue.cpp
+++ b/lldb/source/Core/DumpRegisterValue.cpp
@@ -17,26 +17,10 @@
#include "lldb/Utility/RegisterValue.h"
#include "lldb/Utility/StreamString.h"
#include "lldb/lldb-private-types.h"
-
-#if !defined(__has_builtin)
-#define __has_builtin(x) 0
-#endif
-
-#if __has_builtin(__builtin_bswap32) && __has_builtin(__builtin_bswap64)
-#define bswap_32(x) __builtin_bswap32(x)
-#define bswap_64(x) __builtin_bswap64(x)
-#elif defined(_MSC_VER)
-#define bswap_32(x) _byteswap_ulong(x)
-#define bswap_64(x) _byteswap_uint64(x)
-#else
-#include <byteswap.h>
-#endif
+#include "llvm/ADT/bit.h"
using namespace lldb;
-static uint32_t swap_value(uint32_t v) { return bswap_32(v); }
-static uint64_t swap_value(uint64_t v) { return bswap_64(v); }
-
template <typename T>
static void dump_type_value(lldb_private::CompilerType &fields_type, T value,
lldb_private::ExecutionContextScope *exe_scope,
@@ -55,7 +39,7 @@ static void dump_type_value(lldb_private::CompilerType &fields_type, T value,
// Then we need to match the target's endian on a byte level as well.
if (lldb_private::endian::InlHostByteOrder() != target_order)
- value = swap_value(value);
+ value = llvm::byteswap(value);
lldb_private::DataExtractor data_extractor{
&value, sizeof(T), lldb_private::endian::InlHostByteOrder(), 8};
More information about the lldb-commits
mailing list