[llvm] [NFC] Rename variable to better document usage. (PR #71973)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 10 10:48:12 PST 2023
https://github.com/stephenpeckham created https://github.com/llvm/llvm-project/pull/71973
Change variable DisassembleAsData to DisassembleELFAsData so that its name better matches its usage.
>From fa40f30caf48861de4adbfceba730998b68f8380 Mon Sep 17 00:00:00 2001
From: Stephen Peckham <speckham at us.ibm.com>
Date: Fri, 10 Nov 2023 13:42:23 -0500
Subject: [PATCH] [NFC] Rename variable to better document usage.
---
llvm/tools/llvm-objdump/llvm-objdump.cpp | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/llvm/tools/llvm-objdump/llvm-objdump.cpp b/llvm/tools/llvm-objdump/llvm-objdump.cpp
index a112c50bf7f2715..e5b8e014392c54c 100644
--- a/llvm/tools/llvm-objdump/llvm-objdump.cpp
+++ b/llvm/tools/llvm-objdump/llvm-objdump.cpp
@@ -1780,19 +1780,19 @@ disassembleObject(ObjectFile &Obj, const ObjectFile &DbgObj,
// inside functions, for which STT_FUNC would be inaccurate.
//
// So here, we spot whether there's any non-data symbol present at all,
- // and only set the DisassembleAsData flag if there isn't. Also, we use
+ // and only set the DisassembleELFAsData flag if there isn't. Also, we use
// this distinction to inform the decision of which symbol to print at
// the head of the section, so that if we're printing code, we print a
// code-related symbol name to go with it.
- bool DisassembleAsData = false;
+ bool DisassembleELFAsData = false;
size_t DisplaySymIndex = SymbolsHere.size() - 1;
if (Obj.isELF() && !DisassembleAll && Section.isText()) {
- DisassembleAsData = true; // unless we find a code symbol below
+ DisassembleELFAsData = true; // unless we find a code symbol below
for (size_t i = 0; i < SymbolsHere.size(); ++i) {
uint8_t SymTy = SymbolsHere[i].Type;
if (SymTy != ELF::STT_OBJECT && SymTy != ELF::STT_COMMON) {
- DisassembleAsData = false;
+ DisassembleELFAsData = false;
DisplaySymIndex = i;
}
}
@@ -1943,7 +1943,7 @@ disassembleObject(ObjectFile &Obj, const ObjectFile &DbgObj,
if (SectionAddr < StartAddress)
Index = std::max<uint64_t>(Index, StartAddress - SectionAddr);
- if (DisassembleAsData) {
+ if (DisassembleELFAsData) {
dumpELFData(SectionAddr, Index, End, Bytes);
Index = End;
continue;
More information about the llvm-commits
mailing list