[llvm] r310739 - [opt-viewer] Use Python 3-compatible `intern()`

Brian Gesiak via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 11 10:56:57 PDT 2017

Author: modocache
Date: Fri Aug 11 10:56:57 2017
New Revision: 310739

URL: http://llvm.org/viewvc/llvm-project?rev=310739&view=rev
[opt-viewer] Use Python 3-compatible `intern()`

In Python 2, `intern()` is a builtin function available to all programs.
In Python 3, it was moved into the `sys` module, available as
`sys.intern`. Import it such that, within `optrecord.py`, `intern()` is
available whether run using Python 2 or 3.

Test Plan:
Run `opt-viewer.py` using Python 3, confirm it no longer
encounters a runtime error when `intern()` is called.

Reviewers: anemet

Reviewed By: anemet

Subscribers: llvm-commits

Differential Revision: https://reviews.llvm.org/D36622


Modified: llvm/trunk/tools/opt-viewer/optrecord.py
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/opt-viewer/optrecord.py?rev=310739&r1=310738&r2=310739&view=diff
--- llvm/trunk/tools/opt-viewer/optrecord.py (original)
+++ llvm/trunk/tools/opt-viewer/optrecord.py Fri Aug 11 10:56:57 2017
@@ -17,6 +17,12 @@ import functools
 from multiprocessing import Lock
 import os, os.path
 import subprocess
+    # The previously builtin function `intern()` was moved
+    # to the `sys` module in Python 3.
+    from sys import intern
+    pass
 import optpmap

More information about the llvm-commits mailing list