[llvm] [llvm-debuginfo-analyzer] Add support for WebAssembly binary format. (PR #82588)

via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 21 22:21:43 PST 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 0e8d1877cd145719b7acb707539287b7b877a555 9c9ce3cb5347c31fba5e8ac1480a7525e6d0964f -- llvm/test/tools/llvm-debuginfo-analyzer/WebAssembly/Inputs/definitions.h llvm/test/tools/llvm-debuginfo-analyzer/WebAssembly/Inputs/hello-world.cpp llvm/test/tools/llvm-debuginfo-analyzer/WebAssembly/Inputs/pr-43860.cpp llvm/test/tools/llvm-debuginfo-analyzer/WebAssembly/Inputs/pr-44884.cpp llvm/test/tools/llvm-debuginfo-analyzer/WebAssembly/Inputs/pr-46466.cpp llvm/test/tools/llvm-debuginfo-analyzer/WebAssembly/Inputs/test.cpp llvm/include/llvm/DebugInfo/LogicalView/Readers/LVBinaryReader.h llvm/lib/DebugInfo/LogicalView/LVReaderHandler.cpp llvm/lib/DebugInfo/LogicalView/Readers/LVBinaryReader.cpp llvm/lib/DebugInfo/LogicalView/Readers/LVELFReader.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/llvm/lib/DebugInfo/LogicalView/Readers/LVBinaryReader.cpp b/llvm/lib/DebugInfo/LogicalView/Readers/LVBinaryReader.cpp
index d219b8abe1..871c1bed03 100644
--- a/llvm/lib/DebugInfo/LogicalView/Readers/LVBinaryReader.cpp
+++ b/llvm/lib/DebugInfo/LogicalView/Readers/LVBinaryReader.cpp
@@ -188,11 +188,11 @@ void LVBinaryReader::mapVirtualAddress(const object::ObjectFile &Obj) {
         (*SectionNameOrErr).equals("CODE") ||
         (*SectionNameOrErr).equals(".code"))
       DotTextSectionIndex = Section.getIndex();
-      // If the object is WebAssembly, update the address offset that
-      // will be added to DWARF DW_AT_* attributes.
-      if (Obj.isWasm()) {
-        WasmOffset = Section.getAddress();
-      }
+    // If the object is WebAssembly, update the address offset that
+    // will be added to DWARF DW_AT_* attributes.
+    if (Obj.isWasm()) {
+      WasmOffset = Section.getAddress();
+    }
   }
 
   // Process the symbol table.

``````````

</details>


https://github.com/llvm/llvm-project/pull/82588


More information about the llvm-commits mailing list