[Openmp-commits] [PATCH] D143702: [NFC] Convert openmp/runtime/tools/summarizeStats.py to python 3 syntax

[Jonathan Peyton via Phabricator via Openmp-commits]

jlpeyton added inline comments.


================
Comment at: openmp/runtime/tools/summarizeStats.py:306
                 numThreads = tmp[s]['SampleCount']['Total_OMP_parallel']
-                for key in data.keys():
+                for key in list(data.keys()):
                     if key[0:5] == 'Total':
----------------
serge-sans-paille wrote:
> not needed
I do think this list() is necessary because the `del data[key]` line may change the size of the dictionary during iteration.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143702/new/

https://reviews.llvm.org/D143702