[PATCH] D112075: Use safe yaml load_all

[ChenZheng via Phabricator via llvm-commits]

shchenz added a comment.

Hi @kongyi , this breaks opt-viewer.py usage on the AIX platform.

  opt-viewer/opt-viewer.py  test.opt.yaml --demangler cat
  For faster parsing, you may want to install libYAML for PyYAML
  Reading YAML files...
          1 of 1multiprocessing.pool.RemoteTraceback: 
  """
  Traceback (most recent call last):
    File "/opt/freeware/lib64/python3.7/multiprocessing/pool.py", line 121, in worker
      result = (True, func(*args, **kwds))
    File "/opt/freeware/lib64/python3.7/multiprocessing/pool.py", line 44, in mapstar
      return list(map(*args))
    File "/home/czhengsz/llvm/dev/llvm-project/llvm/tools/opt-viewer/optpmap.py", line 25, in _wrapped_func
      return func(argument, filter_)
    File "/home/czhengsz/llvm/dev/llvm-project/llvm/tools/opt-viewer/optrecord.py", line 277, in get_remarks
      docs = yaml.safe_load_all(f, Loader=Loader)
  TypeError: safe_load_all() got an unexpected keyword argument 'Loader'
  """


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112075