[PATCH] D33095: [analyzer] Avoid allocation in Std C function modelling.

2017-05-12 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL302879: [analyzer] Avoid an allocation in Std C function modelling (authored by xazax). Changed prior to commit: https://reviews.llvm.org/D33095?vs=98610&id=98730#toc Repository: rL LLVM https://rev

[PATCH] D33095: [analyzer] Avoid allocation in Std C function modelling.

2017-05-11 Thread Gábor Horváth via Phabricator via cfe-commits
xazax.hun added a comment. In https://reviews.llvm.org/D33095#752879, @NoQ wrote: > Is it an actual performance problem? Cause i think i did somehow test the > checker for performance regressions and it seemed all good. I did not measure it. Just spotted by skimming through the code. I think,

[PATCH] D33095: [analyzer] Avoid allocation in Std C function modelling.

2017-05-11 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Thanks :o I guess i was planning for C++ support here, but never reached that far. Is it an actual performance problem? Cause i think i did the checker for performance regressions and it seemed all

[PATCH] D33095: [analyzer] Avoid allocation in Std C function modelling.

2017-05-11 Thread Gábor Horváth via Phabricator via cfe-commits
xazax.hun created this revision. Herald added a subscriber: whisperity. Avoid an allocation in modelling Std C functions to improve the performance. Repository: rL LLVM https://reviews.llvm.org/D33095 Files: lib/StaticAnalyzer/Checkers/StdLibraryFunctionsChecker.cpp Index: lib/StaticAnal