This is an archive of the discontinued LLVM Phabricator instance.

Paths

[DWARF] Fix an incorrect format specifier.
ClosedPublic

Authored by ikudrin on Jul 12 2019, 2:15 AM.

Reviewers

dblaikie
JDevlieghere

Commits

Summary

This adjusts the format specifier for PCOffset because it is uint16_t.

This revision is now accepted and ready to land.Jul 12 2019, 8:15 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

llvm/

trunk/

lib/

DebugInfo/

DWARF/

2 lines

Show First 20 Lines • Show All 752 Lines • ▼ Show 20 Lines	if (Opcode == 0) {
// judge when the computation of a special opcode overflows and		// judge when the computation of a special opcode overflows and
// requires the use of DW_LNS_advance_pc. Such assemblers, however,		// requires the use of DW_LNS_advance_pc. Such assemblers, however,
// can use DW_LNS_fixed_advance_pc instead, sacrificing compression.		// can use DW_LNS_fixed_advance_pc instead, sacrificing compression.
{		{
uint16_t PCOffset = DebugLineData.getU16(OffsetPtr);		uint16_t PCOffset = DebugLineData.getU16(OffsetPtr);
State.Row.Address.Address += PCOffset;		State.Row.Address.Address += PCOffset;
if (OS)		if (OS)
*OS		*OS
<< format(" (0x%16.16" PRIx64 ")", PCOffset);		<< format(" (0x%4.4" PRIx16 ")", PCOffset);
}		}
break;		break;

case DW_LNS_set_prologue_end:		case DW_LNS_set_prologue_end:
// Takes no arguments. Set the prologue_end register of the		// Takes no arguments. Set the prologue_end register of the
// state machine to true		// state machine to true
State.Row.PrologueEnd = true;		State.Row.PrologueEnd = true;
break;		break;
▲ Show 20 Lines • Show All 378 Lines • Show Last 20 Lines