[PATCH] Add scan-build python implementation

Devin Coughlin dcoughlin at apple.com
Thu Jun 18 11:13:57 PDT 2015


Thanks for the update! Some additional questions:

- Why does scan-build-py need to distinguish between "installed" and "non-installed mode"? And, in particular, does using the dynamic linker preload technique require installation? It seems to me that it shouldn't.

- Can we just remove "installed mode"? What benefit does it provide? If the only benefit is speed, do you think the speed is worth the extra complexity?

- Does your latest update call the existing perl ccc-analyzer script? I see you have added python-based "intercept-cc" and "intercept-cxx". Can scan-build-py be changed to call the existing ccc-analyzer rather than doing its own wrapping? The goal of deferring to ccc-analyzer is to make sure that we preserve the existing behavior of scan-build.


http://reviews.llvm.org/D9600

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list