[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