[libunwind][NFC] fix typo and wrong debug log. #163314
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
|
@llvm/pr-subscribers-libunwind Author: Wu Yingcong (yingcong-wu) ChangesTypo: Full diff: https://github.com/llvm/llvm-project/pull/163314.diff 2 Files Affected:
diff --git a/libunwind/src/DwarfInstructions.hpp b/libunwind/src/DwarfInstructions.hpp
index e7be0d6d5d635..3dd19f7d94fdc 100644
--- a/libunwind/src/DwarfInstructions.hpp
+++ b/libunwind/src/DwarfInstructions.hpp
@@ -938,7 +938,7 @@ DwarfInstructions<A, R>::evaluateExpression(pint_t expression, A &addressSpace,
case DW_OP_xderef_size:
case DW_OP_nop:
- case DW_OP_push_object_addres:
+ case DW_OP_push_object_address:
case DW_OP_call2:
case DW_OP_call4:
case DW_OP_call_ref:
diff --git a/libunwind/src/dwarf2.h b/libunwind/src/dwarf2.h
index 68ad882347203..17f98d8357ab7 100644
--- a/libunwind/src/dwarf2.h
+++ b/libunwind/src/dwarf2.h
@@ -81,159 +81,158 @@ enum {
// DWARF expressions
enum {
- DW_OP_addr = 0x03, // constant address (size target specific)
- DW_OP_deref = 0x06,
- DW_OP_const1u = 0x08, // 1-byte constant
- DW_OP_const1s = 0x09, // 1-byte constant
- DW_OP_const2u = 0x0A, // 2-byte constant
- DW_OP_const2s = 0x0B, // 2-byte constant
- DW_OP_const4u = 0x0C, // 4-byte constant
- DW_OP_const4s = 0x0D, // 4-byte constant
- DW_OP_const8u = 0x0E, // 8-byte constant
- DW_OP_const8s = 0x0F, // 8-byte constant
- DW_OP_constu = 0x10, // ULEB128 constant
- DW_OP_consts = 0x11, // SLEB128 constant
- DW_OP_dup = 0x12,
- DW_OP_drop = 0x13,
- DW_OP_over = 0x14,
- DW_OP_pick = 0x15, // 1-byte stack index
- DW_OP_swap = 0x16,
- DW_OP_rot = 0x17,
- DW_OP_xderef = 0x18,
- DW_OP_abs = 0x19,
- DW_OP_and = 0x1A,
- DW_OP_div = 0x1B,
- DW_OP_minus = 0x1C,
- DW_OP_mod = 0x1D,
- DW_OP_mul = 0x1E,
- DW_OP_neg = 0x1F,
- DW_OP_not = 0x20,
- DW_OP_or = 0x21,
- DW_OP_plus = 0x22,
- DW_OP_plus_uconst = 0x23, // ULEB128 addend
- DW_OP_shl = 0x24,
- DW_OP_shr = 0x25,
- DW_OP_shra = 0x26,
- DW_OP_xor = 0x27,
- DW_OP_skip = 0x2F, // signed 2-byte constant
- DW_OP_bra = 0x28, // signed 2-byte constant
- DW_OP_eq = 0x29,
- DW_OP_ge = 0x2A,
- DW_OP_gt = 0x2B,
- DW_OP_le = 0x2C,
- DW_OP_lt = 0x2D,
- DW_OP_ne = 0x2E,
- DW_OP_lit0 = 0x30, // Literal 0
- DW_OP_lit1 = 0x31, // Literal 1
- DW_OP_lit2 = 0x32, // Literal 2
- DW_OP_lit3 = 0x33, // Literal 3
- DW_OP_lit4 = 0x34, // Literal 4
- DW_OP_lit5 = 0x35, // Literal 5
- DW_OP_lit6 = 0x36, // Literal 6
- DW_OP_lit7 = 0x37, // Literal 7
- DW_OP_lit8 = 0x38, // Literal 8
- DW_OP_lit9 = 0x39, // Literal 9
- DW_OP_lit10 = 0x3A, // Literal 10
- DW_OP_lit11 = 0x3B, // Literal 11
- DW_OP_lit12 = 0x3C, // Literal 12
- DW_OP_lit13 = 0x3D, // Literal 13
- DW_OP_lit14 = 0x3E, // Literal 14
- DW_OP_lit15 = 0x3F, // Literal 15
- DW_OP_lit16 = 0x40, // Literal 16
- DW_OP_lit17 = 0x41, // Literal 17
- DW_OP_lit18 = 0x42, // Literal 18
- DW_OP_lit19 = 0x43, // Literal 19
- DW_OP_lit20 = 0x44, // Literal 20
- DW_OP_lit21 = 0x45, // Literal 21
- DW_OP_lit22 = 0x46, // Literal 22
- DW_OP_lit23 = 0x47, // Literal 23
- DW_OP_lit24 = 0x48, // Literal 24
- DW_OP_lit25 = 0x49, // Literal 25
- DW_OP_lit26 = 0x4A, // Literal 26
- DW_OP_lit27 = 0x4B, // Literal 27
- DW_OP_lit28 = 0x4C, // Literal 28
- DW_OP_lit29 = 0x4D, // Literal 29
- DW_OP_lit30 = 0x4E, // Literal 30
- DW_OP_lit31 = 0x4F, // Literal 31
- DW_OP_reg0 = 0x50, // Contents of reg0
- DW_OP_reg1 = 0x51, // Contents of reg1
- DW_OP_reg2 = 0x52, // Contents of reg2
- DW_OP_reg3 = 0x53, // Contents of reg3
- DW_OP_reg4 = 0x54, // Contents of reg4
- DW_OP_reg5 = 0x55, // Contents of reg5
- DW_OP_reg6 = 0x56, // Contents of reg6
- DW_OP_reg7 = 0x57, // Contents of reg7
- DW_OP_reg8 = 0x58, // Contents of reg8
- DW_OP_reg9 = 0x59, // Contents of reg9
- DW_OP_reg10 = 0x5A, // Contents of reg10
- DW_OP_reg11 = 0x5B, // Contents of reg11
- DW_OP_reg12 = 0x5C, // Contents of reg12
- DW_OP_reg13 = 0x5D, // Contents of reg13
- DW_OP_reg14 = 0x5E, // Contents of reg14
- DW_OP_reg15 = 0x5F, // Contents of reg15
- DW_OP_reg16 = 0x60, // Contents of reg16
- DW_OP_reg17 = 0x61, // Contents of reg17
- DW_OP_reg18 = 0x62, // Contents of reg18
- DW_OP_reg19 = 0x63, // Contents of reg19
- DW_OP_reg20 = 0x64, // Contents of reg20
- DW_OP_reg21 = 0x65, // Contents of reg21
- DW_OP_reg22 = 0x66, // Contents of reg22
- DW_OP_reg23 = 0x67, // Contents of reg23
- DW_OP_reg24 = 0x68, // Contents of reg24
- DW_OP_reg25 = 0x69, // Contents of reg25
- DW_OP_reg26 = 0x6A, // Contents of reg26
- DW_OP_reg27 = 0x6B, // Contents of reg27
- DW_OP_reg28 = 0x6C, // Contents of reg28
- DW_OP_reg29 = 0x6D, // Contents of reg29
- DW_OP_reg30 = 0x6E, // Contents of reg30
- DW_OP_reg31 = 0x6F, // Contents of reg31
- DW_OP_breg0 = 0x70, // base register 0 + SLEB128 offset
- DW_OP_breg1 = 0x71, // base register 1 + SLEB128 offset
- DW_OP_breg2 = 0x72, // base register 2 + SLEB128 offset
- DW_OP_breg3 = 0x73, // base register 3 + SLEB128 offset
- DW_OP_breg4 = 0x74, // base register 4 + SLEB128 offset
- DW_OP_breg5 = 0x75, // base register 5 + SLEB128 offset
- DW_OP_breg6 = 0x76, // base register 6 + SLEB128 offset
- DW_OP_breg7 = 0x77, // base register 7 + SLEB128 offset
- DW_OP_breg8 = 0x78, // base register 8 + SLEB128 offset
- DW_OP_breg9 = 0x79, // base register 9 + SLEB128 offset
- DW_OP_breg10 = 0x7A, // base register 10 + SLEB128 offset
- DW_OP_breg11 = 0x7B, // base register 11 + SLEB128 offset
- DW_OP_breg12 = 0x7C, // base register 12 + SLEB128 offset
- DW_OP_breg13 = 0x7D, // base register 13 + SLEB128 offset
- DW_OP_breg14 = 0x7E, // base register 14 + SLEB128 offset
- DW_OP_breg15 = 0x7F, // base register 15 + SLEB128 offset
- DW_OP_breg16 = 0x80, // base register 16 + SLEB128 offset
- DW_OP_breg17 = 0x81, // base register 17 + SLEB128 offset
- DW_OP_breg18 = 0x82, // base register 18 + SLEB128 offset
- DW_OP_breg19 = 0x83, // base register 19 + SLEB128 offset
- DW_OP_breg20 = 0x84, // base register 20 + SLEB128 offset
- DW_OP_breg21 = 0x85, // base register 21 + SLEB128 offset
- DW_OP_breg22 = 0x86, // base register 22 + SLEB128 offset
- DW_OP_breg23 = 0x87, // base register 23 + SLEB128 offset
- DW_OP_breg24 = 0x88, // base register 24 + SLEB128 offset
- DW_OP_breg25 = 0x89, // base register 25 + SLEB128 offset
- DW_OP_breg26 = 0x8A, // base register 26 + SLEB128 offset
- DW_OP_breg27 = 0x8B, // base register 27 + SLEB128 offset
- DW_OP_breg28 = 0x8C, // base register 28 + SLEB128 offset
- DW_OP_breg29 = 0x8D, // base register 29 + SLEB128 offset
- DW_OP_breg30 = 0x8E, // base register 30 + SLEB128 offset
- DW_OP_breg31 = 0x8F, // base register 31 + SLEB128 offset
- DW_OP_regx = 0x90, // ULEB128 register
- DW_OP_fbreg = 0x91, // SLEB128 offset
- DW_OP_bregx = 0x92, // ULEB128 register followed by SLEB128 offset
- DW_OP_piece = 0x93, // ULEB128 size of piece addressed
- DW_OP_deref_size = 0x94, // 1-byte size of data retrieved
- DW_OP_xderef_size = 0x95, // 1-byte size of data retrieved
- DW_OP_nop = 0x96,
- DW_OP_push_object_addres = 0x97,
- DW_OP_call2 = 0x98, // 2-byte offset of DIE
- DW_OP_call4 = 0x99, // 4-byte offset of DIE
- DW_OP_call_ref = 0x9A, // 4- or 8-byte offset of DIE
- DW_OP_lo_user = 0xE0,
- DW_OP_APPLE_uninit = 0xF0,
- DW_OP_hi_user = 0xFF
+ DW_OP_addr = 0x03, // constant address (size target specific)
+ DW_OP_deref = 0x06,
+ DW_OP_const1u = 0x08, // 1-byte constant
+ DW_OP_const1s = 0x09, // 1-byte constant
+ DW_OP_const2u = 0x0A, // 2-byte constant
+ DW_OP_const2s = 0x0B, // 2-byte constant
+ DW_OP_const4u = 0x0C, // 4-byte constant
+ DW_OP_const4s = 0x0D, // 4-byte constant
+ DW_OP_const8u = 0x0E, // 8-byte constant
+ DW_OP_const8s = 0x0F, // 8-byte constant
+ DW_OP_constu = 0x10, // ULEB128 constant
+ DW_OP_consts = 0x11, // SLEB128 constant
+ DW_OP_dup = 0x12,
+ DW_OP_drop = 0x13,
+ DW_OP_over = 0x14,
+ DW_OP_pick = 0x15, // 1-byte stack index
+ DW_OP_swap = 0x16,
+ DW_OP_rot = 0x17,
+ DW_OP_xderef = 0x18,
+ DW_OP_abs = 0x19,
+ DW_OP_and = 0x1A,
+ DW_OP_div = 0x1B,
+ DW_OP_minus = 0x1C,
+ DW_OP_mod = 0x1D,
+ DW_OP_mul = 0x1E,
+ DW_OP_neg = 0x1F,
+ DW_OP_not = 0x20,
+ DW_OP_or = 0x21,
+ DW_OP_plus = 0x22,
+ DW_OP_plus_uconst = 0x23, // ULEB128 addend
+ DW_OP_shl = 0x24,
+ DW_OP_shr = 0x25,
+ DW_OP_shra = 0x26,
+ DW_OP_xor = 0x27,
+ DW_OP_skip = 0x2F, // signed 2-byte constant
+ DW_OP_bra = 0x28, // signed 2-byte constant
+ DW_OP_eq = 0x29,
+ DW_OP_ge = 0x2A,
+ DW_OP_gt = 0x2B,
+ DW_OP_le = 0x2C,
+ DW_OP_lt = 0x2D,
+ DW_OP_ne = 0x2E,
+ DW_OP_lit0 = 0x30, // Literal 0
+ DW_OP_lit1 = 0x31, // Literal 1
+ DW_OP_lit2 = 0x32, // Literal 2
+ DW_OP_lit3 = 0x33, // Literal 3
+ DW_OP_lit4 = 0x34, // Literal 4
+ DW_OP_lit5 = 0x35, // Literal 5
+ DW_OP_lit6 = 0x36, // Literal 6
+ DW_OP_lit7 = 0x37, // Literal 7
+ DW_OP_lit8 = 0x38, // Literal 8
+ DW_OP_lit9 = 0x39, // Literal 9
+ DW_OP_lit10 = 0x3A, // Literal 10
+ DW_OP_lit11 = 0x3B, // Literal 11
+ DW_OP_lit12 = 0x3C, // Literal 12
+ DW_OP_lit13 = 0x3D, // Literal 13
+ DW_OP_lit14 = 0x3E, // Literal 14
+ DW_OP_lit15 = 0x3F, // Literal 15
+ DW_OP_lit16 = 0x40, // Literal 16
+ DW_OP_lit17 = 0x41, // Literal 17
+ DW_OP_lit18 = 0x42, // Literal 18
+ DW_OP_lit19 = 0x43, // Literal 19
+ DW_OP_lit20 = 0x44, // Literal 20
+ DW_OP_lit21 = 0x45, // Literal 21
+ DW_OP_lit22 = 0x46, // Literal 22
+ DW_OP_lit23 = 0x47, // Literal 23
+ DW_OP_lit24 = 0x48, // Literal 24
+ DW_OP_lit25 = 0x49, // Literal 25
+ DW_OP_lit26 = 0x4A, // Literal 26
+ DW_OP_lit27 = 0x4B, // Literal 27
+ DW_OP_lit28 = 0x4C, // Literal 28
+ DW_OP_lit29 = 0x4D, // Literal 29
+ DW_OP_lit30 = 0x4E, // Literal 30
+ DW_OP_lit31 = 0x4F, // Literal 31
+ DW_OP_reg0 = 0x50, // Contents of reg0
+ DW_OP_reg1 = 0x51, // Contents of reg1
+ DW_OP_reg2 = 0x52, // Contents of reg2
+ DW_OP_reg3 = 0x53, // Contents of reg3
+ DW_OP_reg4 = 0x54, // Contents of reg4
+ DW_OP_reg5 = 0x55, // Contents of reg5
+ DW_OP_reg6 = 0x56, // Contents of reg6
+ DW_OP_reg7 = 0x57, // Contents of reg7
+ DW_OP_reg8 = 0x58, // Contents of reg8
+ DW_OP_reg9 = 0x59, // Contents of reg9
+ DW_OP_reg10 = 0x5A, // Contents of reg10
+ DW_OP_reg11 = 0x5B, // Contents of reg11
+ DW_OP_reg12 = 0x5C, // Contents of reg12
+ DW_OP_reg13 = 0x5D, // Contents of reg13
+ DW_OP_reg14 = 0x5E, // Contents of reg14
+ DW_OP_reg15 = 0x5F, // Contents of reg15
+ DW_OP_reg16 = 0x60, // Contents of reg16
+ DW_OP_reg17 = 0x61, // Contents of reg17
+ DW_OP_reg18 = 0x62, // Contents of reg18
+ DW_OP_reg19 = 0x63, // Contents of reg19
+ DW_OP_reg20 = 0x64, // Contents of reg20
+ DW_OP_reg21 = 0x65, // Contents of reg21
+ DW_OP_reg22 = 0x66, // Contents of reg22
+ DW_OP_reg23 = 0x67, // Contents of reg23
+ DW_OP_reg24 = 0x68, // Contents of reg24
+ DW_OP_reg25 = 0x69, // Contents of reg25
+ DW_OP_reg26 = 0x6A, // Contents of reg26
+ DW_OP_reg27 = 0x6B, // Contents of reg27
+ DW_OP_reg28 = 0x6C, // Contents of reg28
+ DW_OP_reg29 = 0x6D, // Contents of reg29
+ DW_OP_reg30 = 0x6E, // Contents of reg30
+ DW_OP_reg31 = 0x6F, // Contents of reg31
+ DW_OP_breg0 = 0x70, // base register 0 + SLEB128 offset
+ DW_OP_breg1 = 0x71, // base register 1 + SLEB128 offset
+ DW_OP_breg2 = 0x72, // base register 2 + SLEB128 offset
+ DW_OP_breg3 = 0x73, // base register 3 + SLEB128 offset
+ DW_OP_breg4 = 0x74, // base register 4 + SLEB128 offset
+ DW_OP_breg5 = 0x75, // base register 5 + SLEB128 offset
+ DW_OP_breg6 = 0x76, // base register 6 + SLEB128 offset
+ DW_OP_breg7 = 0x77, // base register 7 + SLEB128 offset
+ DW_OP_breg8 = 0x78, // base register 8 + SLEB128 offset
+ DW_OP_breg9 = 0x79, // base register 9 + SLEB128 offset
+ DW_OP_breg10 = 0x7A, // base register 10 + SLEB128 offset
+ DW_OP_breg11 = 0x7B, // base register 11 + SLEB128 offset
+ DW_OP_breg12 = 0x7C, // base register 12 + SLEB128 offset
+ DW_OP_breg13 = 0x7D, // base register 13 + SLEB128 offset
+ DW_OP_breg14 = 0x7E, // base register 14 + SLEB128 offset
+ DW_OP_breg15 = 0x7F, // base register 15 + SLEB128 offset
+ DW_OP_breg16 = 0x80, // base register 16 + SLEB128 offset
+ DW_OP_breg17 = 0x81, // base register 17 + SLEB128 offset
+ DW_OP_breg18 = 0x82, // base register 18 + SLEB128 offset
+ DW_OP_breg19 = 0x83, // base register 19 + SLEB128 offset
+ DW_OP_breg20 = 0x84, // base register 20 + SLEB128 offset
+ DW_OP_breg21 = 0x85, // base register 21 + SLEB128 offset
+ DW_OP_breg22 = 0x86, // base register 22 + SLEB128 offset
+ DW_OP_breg23 = 0x87, // base register 23 + SLEB128 offset
+ DW_OP_breg24 = 0x88, // base register 24 + SLEB128 offset
+ DW_OP_breg25 = 0x89, // base register 25 + SLEB128 offset
+ DW_OP_breg26 = 0x8A, // base register 26 + SLEB128 offset
+ DW_OP_breg27 = 0x8B, // base register 27 + SLEB128 offset
+ DW_OP_breg28 = 0x8C, // base register 28 + SLEB128 offset
+ DW_OP_breg29 = 0x8D, // base register 29 + SLEB128 offset
+ DW_OP_breg30 = 0x8E, // base register 30 + SLEB128 offset
+ DW_OP_breg31 = 0x8F, // base register 31 + SLEB128 offset
+ DW_OP_regx = 0x90, // ULEB128 register
+ DW_OP_fbreg = 0x91, // SLEB128 offset
+ DW_OP_bregx = 0x92, // ULEB128 register followed by SLEB128 offset
+ DW_OP_piece = 0x93, // ULEB128 size of piece addressed
+ DW_OP_deref_size = 0x94, // 1-byte size of data retrieved
+ DW_OP_xderef_size = 0x95, // 1-byte size of data retrieved
+ DW_OP_nop = 0x96,
+ DW_OP_push_object_address = 0x97,
+ DW_OP_call2 = 0x98, // 2-byte offset of DIE
+ DW_OP_call4 = 0x99, // 4-byte offset of DIE
+ DW_OP_call_ref = 0x9A, // 4- or 8-byte offset of DIE
+ DW_OP_lo_user = 0xE0,
+ DW_OP_APPLE_uninit = 0xF0,
+ DW_OP_hi_user = 0xFF
};
-
#endif
|
yingcong-wu
changed the title
MacDue
reviewed
Dec 6, 2025
| // DWARF expressions | ||
| enum { | ||
| DW_OP_addr = 0x03, // constant address (size target specific) | ||
| DW_OP_deref = 0x06, |
Member
There was a problem hiding this comment.
This change should not reformat this entire enum.
Contributor
Author
There was a problem hiding this comment.
I didn't do that, but the git-clang-format thinks otherwise.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Typo:
DW_OP_push_object_addres->DW_OP_push_object_address.Wrong log for
DW_OP_shrandDW_OP_shra.