[PATCH] D51926: [scan-build-py] Prevent crashes of CTU analysis by suppressing warnings

2018-09-20 Thread George Karpenkov via Phabricator via cfe-commits
george.karpenkov accepted this revision.
george.karpenkov added a comment.
This revision is now accepted and ready to land.

If that helps you, then sure.

I'm not sure  I understand why having warnings causes the collection process to 
fail, but I guess ultimately it's not important.


Repository:
  rC Clang

https://reviews.llvm.org/D51926



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


[PATCH] D51926: [scan-build-py] Prevent crashes of CTU analysis by suppressing warnings

2018-09-20 Thread Lorinc Balog via Phabricator via cfe-commits
lorincbalog added a comment.

Yes, `-w` suppresses all warnings (without raising an error) regardless of the 
options' sequence, even if `-Wall -Werror` are present,.


Repository:
  rC Clang

https://reviews.llvm.org/D51926



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


[PATCH] D51926: [scan-build-py] Prevent crashes of CTU analysis by suppressing warnings

2018-09-11 Thread Whisperity via Phabricator via cfe-commits
whisperity added a comment.

Will this properly synergise across compilers with user-specified warning 
options, such as `-Wall -Werror`?


Repository:
  rC Clang

https://reviews.llvm.org/D51926



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


[PATCH] D51926: [scan-build-py] Prevent crashes of CTU analysis by suppressing warnings

2018-09-11 Thread Lorinc Balog via Phabricator via cfe-commits
lorincbalog created this revision.
lorincbalog added reviewers: rizsotto.mailinglist, dcoughlin.
Herald added subscribers: cfe-commits, whisperity.

During the mapping of functions for cross-translation unit analysis, the 
functions' names and the containing files are collected by clang-func-mapping. 
Warnings are stored in the collection too, which later makes the parsing of the 
collection fail. Proposed solution is to suppress all warnings during the 
function mapping.


Repository:
  rC Clang

https://reviews.llvm.org/D51926

Files:
  tools/scan-build-py/libscanbuild/analyze.py


Index: tools/scan-build-py/libscanbuild/analyze.py
===
--- tools/scan-build-py/libscanbuild/analyze.py
+++ tools/scan-build-py/libscanbuild/analyze.py
@@ -598,6 +598,7 @@
 funcmap_command.append(opts['file'])
 funcmap_command.append('--')
 funcmap_command.extend(args)
+funcmap_command.append('-w')
 logging.debug("Generating function map using '%s'", funcmap_command)
 func_src_list = run_command(funcmap_command, cwd=opts['directory'])
 func_ast_list = func_map_list_src_to_ast(func_src_list)


Index: tools/scan-build-py/libscanbuild/analyze.py
===
--- tools/scan-build-py/libscanbuild/analyze.py
+++ tools/scan-build-py/libscanbuild/analyze.py
@@ -598,6 +598,7 @@
 funcmap_command.append(opts['file'])
 funcmap_command.append('--')
 funcmap_command.extend(args)
+funcmap_command.append('-w')
 logging.debug("Generating function map using '%s'", funcmap_command)
 func_src_list = run_command(funcmap_command, cwd=opts['directory'])
 func_ast_list = func_map_list_src_to_ast(func_src_list)
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits