[llvm] [DebugInfo][DWARF] Utilize DW_AT_LLVM_stmt_sequence attr in line table lookups (PR #123391)

via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 17 12:04:00 PST 2025


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 7d8b4eb0ead277f41ff69525ed807f9f6e227f37 82a6fb89d00856196f534383115afe08b42fe417 --extensions cpp,h -- llvm/include/llvm/DebugInfo/DWARF/DWARFDebugLine.h llvm/lib/DebugInfo/DWARF/DWARFDebugLine.cpp llvm/unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp
``````````

</details>

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

``````````diff
diff --git a/llvm/unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp b/llvm/unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp
index 7a1d634900..b68944fb7f 100644
--- a/llvm/unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp
+++ b/llvm/unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp
@@ -2035,18 +2035,22 @@ TEST_F(DebugLineBasicFixture, PrintPathsProperly) {
   EXPECT_THAT(Result.c_str(), MatchesRegex("a dir.b dir.b file"));
 }
 
-/// Test that lookupAddressRange correctly filters rows based on DW_AT_LLVM_stmt_sequence.
+/// Test that lookupAddressRange correctly filters rows based on
+/// DW_AT_LLVM_stmt_sequence.
 ///
 /// This test verifies that:
-/// 1. When a DIE has a DW_AT_LLVM_stmt_sequence attribute, lookupAddressRange only returns 
+/// 1. When a DIE has a DW_AT_LLVM_stmt_sequence attribute, lookupAddressRange
+/// only returns
 ///    rows from the sequence starting at the specified offset
-/// 2. When a DIE has an invalid DW_AT_LLVM_stmt_sequence offset, no rows are returned
-/// 3. When no DW_AT_LLVM_stmt_sequence is present, all matching rows are returned
+/// 2. When a DIE has an invalid DW_AT_LLVM_stmt_sequence offset, no rows are
+/// returned
+/// 3. When no DW_AT_LLVM_stmt_sequence is present, all matching rows are
+/// returned
 ///
-/// The test creates a line table with two sequences at the same address range but 
-/// different line numbers. It then creates three subprogram DIEs:
+/// The test creates a line table with two sequences at the same address range
+/// but different line numbers. It then creates three subprogram DIEs:
 /// - One with DW_AT_LLVM_stmt_sequence pointing to the first sequence
-/// - One with DW_AT_LLVM_stmt_sequence pointing to the second sequence  
+/// - One with DW_AT_LLVM_stmt_sequence pointing to the second sequence
 /// - One with an invalid DW_AT_LLVM_stmt_sequence offset
 TEST_F(DebugLineBasicFixture, LookupAddressRangeWithStmtSequenceOffset) {
   if (!setupGenerator())

``````````

</details>


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


More information about the llvm-commits mailing list