Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline

Differential D116680

[4/4][llvm-objdump][RISCV] Support --symbolize-operands for RISC-V
AcceptedPublic
Actions

Authored by asb on Jan 5 2022, 10:36 AM.

Details

Reviewers
jhenderson
MaskRay
craig.topper
luismarques

Diff Detail

Event Timeline

asb created this revision.Jan 5 2022, 10:36 AM
Herald added a reviewer: jhenderson. · View Herald TranscriptJan 5 2022, 10:36 AM
Herald added a reviewer: MaskRay. · View Herald Transcript
Herald added subscribers: VincentWu, luke957, achieveartificialintelligence and 27 others. · View Herald Transcript
asb requested review of this revision.Jan 5 2022, 10:36 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 5 2022, 10:36 AM
Herald added a subscriber: StephenFan. · View Herald Transcript
asb added a parent revision: D116678: [3/4][RISCV][MC] Teach MCInstrAnalysis::evaluateBranch to determine the target of an auipc+jalr.Jan 5 2022, 10:37 AM
asb added reviewers: craig.topper, luismarques.Jan 5 2022, 10:40 AM
Harbormaster completed remote builds in B141718: Diff 397637.Jan 5 2022, 12:02 PM
jhenderson added inline comments.Jan 6 2022, 12:17 AM
llvm/docs/CommandGuide/llvm-objdump.rst
229

I'd probably add a comma after "objects".

asb updated this revision to Diff 397820.Jan 6 2022, 2:11 AM

Address review comment.

asb marked an inline comment as done.Jan 6 2022, 2:11 AM
Harbormaster completed remote builds in B141854: Diff 397820.Jan 6 2022, 2:47 AM
MaskRay accepted this revision.Jan 6 2022, 6:49 PM

LGTM.

This revision is now accepted and ready to land.Jan 6 2022, 6:49 PM
asb updated this revision to Diff 398430.Jan 9 2022, 5:20 AM

Rebase.

Harbormaster completed remote builds in B142302: Diff 398430.Jan 9 2022, 6:52 AM
jhenderson accepted this revision.Jan 10 2022, 1:03 AM
dtcxzyw added a subscriber: dtcxzyw.Mar 10 2023, 4:07 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 10 2023, 4:07 AM
Herald added subscribers: luke, wingo, pmatos and 4 others. · View Herald Transcript

Revision Contents
Changeset List

PathSize
llvm/
docs/
CommandGuide/
2 lines
test/
tools/
llvm-objdump/
ELF/
RISCV/
82 lines
tools/
llvm-objdump/
5 lines

Diff 398430

View Options

llvm/docs/CommandGuide/llvm-objdump.rst

Loading...
View Options

llvm/test/tools/llvm-objdump/ELF/RISCV/branches.s

Loading...
View Options

llvm/tools/llvm-objdump/llvm-objdump.cpp

Loading...