Re: [analyzer] Add support for __builtin_constant_p to BuiltinFunctionChecker

2018-01-24 Thread George Karpenkov via cfe-commits
Hi Felix, If you would like to get the patch reviewed and committed the best way is to create a review in phabricator (http://reviews.llvm.org) and add Devin Coughlin, me, and Artem Dergachev as reviewers. > On Jan 16, 2018, at 7:30 PM, Vedant Kumar wrote: > > + Devin and George > >> On Jan

Re: [analyzer] Add support for __builtin_constant_p to BuiltinFunctionChecker

2018-01-16 Thread Vedant Kumar via cfe-commits
+ Devin and George > On Jan 14, 2018, at 10:44 AM, Felix Kostenzer via cfe-commits > wrote: > > > Hi, > > Added evaluation of __builtin_constant_p to the dedicated StaticAnalyzer > checker > along with a regression test. > > 'make clang-test' results: > > Testing Time: 335.63s > Expected