This is an archive of the discontinued LLVM Phabricator instance.

Symbolize operand bundle blocks for bcanalyzer
ClosedPublic

Authored by sanjoy on Apr 25 2016, 10:55 PM.

Diff Detail

Event Timeline

sanjoy updated this revision to Diff 54966.Apr 25 2016, 10:55 PM
sanjoy retitled this revision from to Symbolize operand bundle blocks for bcanalyzer.
sanjoy updated this object.
sanjoy added a reviewer: mehdi_amini.
sanjoy added a subscriber: llvm-commits.
mehdi_amini accepted this revision.Apr 25 2016, 10:57 PM
mehdi_amini edited edge metadata.

LGTM Thanks!

tools/llvm-bcanalyzer/llvm-bcanalyzer.cpp
108

Not sure if the switch order matches the enum?

This revision is now accepted and ready to land.Apr 25 2016, 10:57 PM
This revision was automatically updated to reflect the committed changes.
sanjoy marked an inline comment as done.
sanjoy added inline comments.Apr 25 2016, 11:05 PM
tools/llvm-bcanalyzer/llvm-bcanalyzer.cpp
108

Followup (as discussed on IRC) - leaving as is, since the switch wasn't sorted to begin with.