This is an archive of the discontinued LLVM Phabricator instance.

[NFC][Py Reformat] Reformat python files in llvm-test-suite
ClosedPublic

Authored by MatzeB on Jun 1 2023, 11:22 AM.

Details

Summary

This is an ongoing series of commits that are reformatting our Python code.

Reformatting is done with black.

If you end up having problems merging this commit because you have made changes to a python file, the best way to handle that is to run git checkout --ours <yourfile> and then reformat it with black.

If you run into any problems, post to discourse about it and we will try to help.

RFC Thread below:

https://discourse.llvm.org/t/rfc-document-and-standardize-python-code-style

Specific to this change:

  • Does not change third-party code in MicroBenchmarks/libs/benchmark.

Fix code to make it parseable by python3/black in the first place:

  • Fixed space/tab confusion in CollectDebugInfoUsingLLDB.py.
  • Change to python3 print syntax: ABI-Testsuite/sample.py, ABI-Testsuite/test/lit.site.cfg, tools/get-report-time

Diff Detail

Repository
rT test-suite

Event Timeline

MatzeB created this revision.Jun 1 2023, 11:22 AM
MatzeB requested review of this revision.Jun 1 2023, 11:22 AM
MatzeB updated this revision to Diff 527537.Jun 1 2023, 11:25 AM
MatzeB edited the summary of this revision. (Show Details)
thieta accepted this revision.Jun 1 2023, 11:58 AM

Thanks!!

This revision is now accepted and ready to land.Jun 1 2023, 11:58 AM
This revision was automatically updated to reflect the committed changes.