This is an archive of the discontinued LLVM Phabricator instance.

[opt-viewer] Listify `dict_items` for Py3 indexing
ClosedPublic

Authored by modocache on Aug 11 2017, 11:47 AM.

Details

Summary

In Python 2, calling dict.items() returns an indexable list, whereas
on Python 3 it returns a set-like dict_items object, which cannot be
indexed. Explicitly onvert the dict_items object so that it can be
indexed when using Python 3.

In combination with D36622, D36623, and D36624, this change allows
opt-viewer.py to exit successfully when run with Python 3.4.

Test Plan:
Run opt-viewer.py using Python 3.4 and confirm it does not encounter a
runtime error when when indexing into dict.items().

Event Timeline

modocache created this revision.Aug 11 2017, 11:47 AM
anemet accepted this revision.Aug 11 2017, 2:45 PM

LGTM, thanks.

This revision is now accepted and ready to land.Aug 11 2017, 2:45 PM

Great, thanks!

modocache closed this revision.Aug 13 2017, 9:17 PM