Author: Alexander Ziaee Date: 2025-06-11T13:27:23+02:00 New Revision: 44a7ecd1d7485be94d3a92021c650175f100d2f7
URL: https://github.com/llvm/llvm-project/commit/44a7ecd1d7485be94d3a92021c650175f100d2f7 DIFF: https://github.com/llvm/llvm-project/commit/44a7ecd1d7485be94d3a92021c650175f100d2f7.diff LOG: [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! Added: Modified: lld/ELF/Relocations.cpp lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerCommon.cpp lldb/tools/debugserver/source/RNBRemote.cpp llvm/lib/Target/X86/X86TargetTransformInfo.cpp openmp/tools/archer/ompt-tsan.cpp Removed: ################################################################################ diff --git a/lld/ELF/Relocations.cpp b/lld/ELF/Relocations.cpp index 8413d8bb2437c..1af01e7247dce 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 67ba42f33d1dd..4a1117222f34c 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<std::string> GDBRemoteCommunicationServerCommon::HandleFeatures( const llvm::ArrayRef<llvm::StringRef> 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 af3c66c71c77e..391d1c50168ea 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 33c9edd24646b..a1a177528eb23 100644 --- a/llvm/lib/Target/X86/X86TargetTransformInfo.cpp +++ b/llvm/lib/Target/X86/X86TargetTransformInfo.cpp @@ -123,7 +123,7 @@ std::optional<unsigned> 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<unsigned> 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 bb60fc6b603f4..c315999af4328 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) _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
