From 44a7ecd1d7485be94d3a92021c650175f100d2f7 Mon Sep 17 00:00:00 2001 From: Alexander Ziaee Date: Wed, 11 Jun 2025 07:27:23 -0400 Subject: [PATCH] [doc] Use ISO nomenclature for 1024 byte units (#133148) Increase specificity by using the correct unit sizes. KBytes is an abbreviation for kB, 1000 bytes, and the hardware industry as well as several operating systems have now switched to using 1000 byte kBs. If this change is acceptable, sometimes GitHub mangles merges to use the original email of the account. $dayjob asks contributions have my work email. Thanks! --- lld/ELF/Relocations.cpp | 2 +- .../Process/gdb-remote/GDBRemoteCommunicationServerCommon.cpp | 2 +- lldb/tools/debugserver/source/RNBRemote.cpp | 2 +- llvm/lib/Target/X86/X86TargetTransformInfo.cpp | 4 ++-- openmp/tools/archer/ompt-tsan.cpp | 2 +- 5 files changed, 6 insertions(+), 6 deletions(-) diff --git a/lld/ELF/Relocations.cpp b/lld/ELF/Relocations.cpp index 8413d8bb2437..1af01e7247dc 100644 --- a/lld/ELF/Relocations.cpp +++ b/lld/ELF/Relocations.cpp @@ -161,7 +161,7 @@ static RelType getMipsPairType(RelType type, bool isLocal) { // symbol, the R_MIPS_GOT16 relocation creates a GOT entry to hold // the high 16 bits of the symbol's value. A paired R_MIPS_LO16 // relocations handle low 16 bits of the address. That allows - // to allocate only one GOT entry for every 64 KBytes of local data. + // to allocate only one GOT entry for every 64 KiB of local data. return isLocal ? R_MIPS_LO16 : R_MIPS_NONE; case R_MICROMIPS_GOT16: return isLocal ? R_MICROMIPS_LO16 : R_MIPS_NONE; diff --git a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerCommon.cpp b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerCommon.cpp index 67ba42f33d1d..4a1117222f34 100644 --- a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerCommon.cpp +++ b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerCommon.cpp @@ -1377,7 +1377,7 @@ GDBRemoteCommunicationServerCommon::GetModuleInfo(llvm::StringRef module_path, std::vector GDBRemoteCommunicationServerCommon::HandleFeatures( const llvm::ArrayRef client_features) { - // 128KBytes is a reasonable max packet size--debugger can always use less. + // 128 KiB is a reasonable max packet size--debugger can always use less. constexpr uint32_t max_packet_size = 128 * 1024; // Features common to platform server and llgs. diff --git a/lldb/tools/debugserver/source/RNBRemote.cpp b/lldb/tools/debugserver/source/RNBRemote.cpp index af3c66c71c77..391d1c50168e 100644 --- a/lldb/tools/debugserver/source/RNBRemote.cpp +++ b/lldb/tools/debugserver/source/RNBRemote.cpp @@ -3476,7 +3476,7 @@ static bool GetProcessNameFrom_vAttach(const char *&p, } rnb_err_t RNBRemote::HandlePacket_qSupported(const char *p) { - uint32_t max_packet_size = 128 * 1024; // 128KBytes is a reasonable max packet + uint32_t max_packet_size = 128 * 1024; // 128 KiB is a reasonable max packet // size--debugger can always use less std::stringstream reply; reply << "qXfer:features:read+;PacketSize=" << std::hex << max_packet_size diff --git a/llvm/lib/Target/X86/X86TargetTransformInfo.cpp b/llvm/lib/Target/X86/X86TargetTransformInfo.cpp index 33c9edd24646..a1a177528eb2 100644 --- a/llvm/lib/Target/X86/X86TargetTransformInfo.cpp +++ b/llvm/lib/Target/X86/X86TargetTransformInfo.cpp @@ -123,7 +123,7 @@ std::optional X86TTIImpl::getCacheSize( // - Broadwell // - Skylake // - Kabylake - return 32 * 1024; // 32 KByte + return 32 * 1024; // 32 KiB case TargetTransformInfo::CacheLevel::L2D: // - Penryn // - Nehalem @@ -134,7 +134,7 @@ std::optional X86TTIImpl::getCacheSize( // - Broadwell // - Skylake // - Kabylake - return 256 * 1024; // 256 KByte + return 256 * 1024; // 256 KiB } llvm_unreachable("Unknown TargetTransformInfo::CacheLevel"); diff --git a/openmp/tools/archer/ompt-tsan.cpp b/openmp/tools/archer/ompt-tsan.cpp index bb60fc6b603f..c315999af432 100644 --- a/openmp/tools/archer/ompt-tsan.cpp +++ b/openmp/tools/archer/ompt-tsan.cpp @@ -1224,7 +1224,7 @@ static void ompt_tsan_finalize(ompt_data_t *tool_data) { if (archer_flags->print_max_rss) { struct rusage end; getrusage(RUSAGE_SELF, &end); - printf("MAX RSS[KBytes] during execution: %ld\n", end.ru_maxrss); + printf("MAX RSS[KiB] during execution: %ld\n", end.ru_maxrss); } if (archer_flags)