[PATCH] D81601: [analyzer] SATest: Use logger in single-threaded mode as well

Valeriy Savchenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 18 08:07:13 PDT 2020


vsavchenko marked an inline comment as done.
vsavchenko added inline comments.


================
Comment at: clang/utils/analyzer/SATestBuild.py:650
 
                 Logger = logging.getLogger(test_info.project.name)
                 LOCAL.stdout = StreamToLogger(Logger, logging.INFO)
----------------
NoQ wrote:
> Now that you have a global `Logger`, is this still relevant?
Yes, it prints the name of the project with the actual message, so it's visible what is this line about


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81601





More information about the cfe-commits mailing list