en Copyright 2015 Java http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist#7a14244c [NFC][clang] Inclusive language: replace masterPort with mainPort[NFC] This patch replaces `masterPort` with `mainPort` in thesetestcases.Reviewed By: ZarkoCADifferential Revision: https://reviews.llvm.org/D113505 List of files: /llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist Tue, 09 Nov 2021 14:38:50 +0000 Quinn Pham <Quinn.Pham@ibm.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist#e2739180 [analyzer] Track leaking object through storesSince we can report memory leaks on one variable, while the originallyallocated object was stored into another one, we should explainhow did it get there.rdar://76645710Differential Revision: https://reviews.llvm.org/D100852 List of files: /llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist Tue, 20 Apr 2021 14:08:55 +0000 Valeriy Savchenko <vsavchenko@apple.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist#61ae2db2 [analyzer] Adjust the reported variable name in retain count checkerWhen reporting leaks, we try to attach the leaking object to somevariable, so it's easier to understand. Before the patch, we alwaystried to use the first variable that stored the object in question.This can get very confusing for the user, if that variable doesn'tcontain that object at the moment of the actual leak. In many cases,the warning is dismissed as false positive and it is effectively afalse positive when we fail to properly explain the warning to theuser.This patch addresses the bigest issue in cases like this. Now wecheck if the variable still contains the leaking symbolic object.If not, we look for the last variable to actually hold it and usethat variable instead.rdar://76645710Differential Revision: https://reviews.llvm.org/D100839 List of files: /llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist Fri, 16 Apr 2021 18:10:05 +0000 Valeriy Savchenko <vsavchenko@apple.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist#5192783b [analyzer][RetainCount] Tie diagnostics to osx.cocoa.RetainCount rather then RetainCountBase, for the most partSimilarly to other patches of mine, I'm trying to uniformize the checkerinterface so that dependency checkers don't emit diagnostics. The checker thatmade me most anxious so far was definitely RetainCount, because it is definitelyimpacted by backward compatibility concerns, and implements a checker hierarchythat is a lot different to other examples of similar size. Also, I don't haveauthority, nor expertise regarding ObjC related code, so I welcome anyobjection/discussion!Differential Revision: https://reviews.llvm.org/D78099 List of files: /llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist Mon, 13 Apr 2020 18:51:27 +0000 Kirstóf Umann <dkszelethus@gmail.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist#15e3e952 [analyzer][NFC][tests] Pre-normalize expected-plistsAs suggested in the review for D62949, this patch pre-normalizes thereference expected output plist files by removing lines containingfields for which we expect differences that should be ignored.llvm-svn: 362877 List of files: /llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist Sat, 08 Jun 2019 13:51:37 +0000 Hubert Tong <hubert.reinterpretcast@gmail.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist#4b0184b2 [analyzer] ConditionBRVisitor: Enhance to write out more informationSummary:Add extra messages to the bug report to inform the user why the analyzer`Taking true/false branch`.Reviewers: NoQ, george.karpenkovReviewed By: NoQSubscribers: gerazo, gsd, dkrupp, whisperity, baloghadamsoftware, xazax.hun, eraman, szepet, a.sidorin, mikhail.ramalho, Szelethus, donat.nagy, cfe-commitsTags: #clangDifferential Revision: https://reviews.llvm.org/D53076llvm-svn: 362020 List of files: /llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist Wed, 29 May 2019 20:06:09 +0000 Csaba Dabis <dabis.csaba98@gmail.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist#0b4699b9 [analyzer][tests][NFC] Add EOF newlines, normalize reference expected filesReference expected files not ending with a newline are normalized tohave said newlines. Additionally `plist-macros-with-expansion.cpp.plist`is modified to add a line that is ignored by `%diff_plist`, but not bythe more sensitive pattern proposed byhttp://lists.llvm.org/pipermail/cfe-dev/2019-April/061904.html for`%normalize_plist`.llvm-svn: 359692 List of files: /llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist Wed, 01 May 2019 15:57:00 +0000 Hubert Tong <hubert.reinterpretcast@gmail.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist#77eae6d4 [analyzer] [RetainCountChecker] Bugfix for tracking top-level parameters of Objective-C methodsDifferential Revision: https://reviews.llvm.org/D57433llvm-svn: 352588 List of files: /llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist Wed, 30 Jan 2019 02:11:04 +0000 George Karpenkov <ekarpenkov@apple.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist#d37ff4e8 [analyzer] [RetainCountChecker] Track input parameters to the top-level functionTrack them for ISL/OS objects by default, and for NS/CF under a flag.rdar://47536377Differential Revision: https://reviews.llvm.org/D57356llvm-svn: 352534 List of files: /llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist Tue, 29 Jan 2019 19:29:59 +0000 George Karpenkov <ekarpenkov@apple.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist#8fd74ebf [analyzer] Reimplement dependencies between checkersUnfortunately, up until now, the fact that certain checkers depended on oneanother was known, but how these actually unfolded was hidden deep within theimplementation. For example, many checkers (like RetainCount, Malloc or CString)modelled a certain functionality, and exposed certain reportable bug types tothe user. For example, while MallocChecker models many many different types ofmemory handling, the actual "unix.MallocChecker" checker the user was exposed towas merely and option to this modeling part.Other than this being an ugly mess, this issue made resolving the checker namingissue almost impossible. (The checker naming issue being that if a checkerregistered more than one checker within its registry function, both checkerobject recieved the same name) Also, if the user explicitly disabled a checkerthat was a dependency of another that _was_ explicitly enabled, it implicitly,without "telling" the user, reenabled it.Clearly, changing this to a well structured, declarative form, where thehandling of dependencies are done on a higher level is very much preferred.This patch, among the detailed things later, makes checkers declare theirdependencies within the TableGen file Checkers.td, and exposes the samefunctionality to plugins and statically linked non-generated checkers throughCheckerRegistry::addDependency. CheckerRegistry now resolves these dependencies,makes sure that checkers are added to CheckerManager in the correct order,and makes sure that if a dependency is disabled, so will be every checker thatdepends on it.In detail:* Add a new field to the Checker class in CheckerBase.td called Dependencies,which is a list of Checkers.* Move unix checkers before cplusplus, as there is no forward declaration intblgen :/* Add the following new checkers: - StackAddrEscapeBase - StackAddrEscapeBase - CStringModeling - DynamicMemoryModeling (base of the MallocChecker family) - IteratorModeling (base of the IteratorChecker family) - ValistBase - SecuritySyntaxChecker (base of bcmp, bcopy, etc...) - NSOrCFErrorDerefChecker (base of NSErrorChecker and CFErrorChecker) - IvarInvalidationModeling (base of IvarInvalidation checker family) - RetainCountBase (base of RetainCount and OSObjectRetainCount)* Clear up and registry functions in MallocChecker, happily remove old FIXMEs.* Add a new addDependency function to CheckerRegistry.* Neatly format RUN lines in files I looked at while debugging.Big thanks to Artem Degrachev for all the guidance through this project!Differential Revision: https://reviews.llvm.org/D54438llvm-svn: 352287 List of files: /llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist Sat, 26 Jan 2019 20:06:54 +0000 Kristof Umann <dkszelethus@gmail.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist#5be959c8 [analyzer] Support for OSObjects out parameters in RetainCountCheckerrdar://46357478rdar://47121327Differential Revision: https://reviews.llvm.org/D56240llvm-svn: 350982 List of files: /llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist Fri, 11 Jan 2019 23:35:17 +0000 George Karpenkov <ekarpenkov@apple.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist#dfd6f7db [analyzer] [NFC] Fix the FIXME in testsllvm-svn: 350946 List of files: /llvm-project-15.0.7/clang/test/Analysis/Inputs/expected-plists/retain-release.m.objcpp.plist Fri, 11 Jan 2019 18:12:49 +0000 George Karpenkov <ekarpenkov@apple.com>