[PATCH] D80522: [Analyzer] [NFC] Parameter Regions
Balogh, Ádám via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jun 3 04:21:01 PDT 2020
baloghadamsoftware marked 2 inline comments as done.
baloghadamsoftware added inline comments.
================
Comment at: clang/include/clang/StaticAnalyzer/Core/PathSensitive/MemRegion.h:921
+public:
+ const VarDecl *getDecl() const override = 0;
+
----------------
NoQ wrote:
> What's the point of overriding one pure virtual method with another pure virtual method?
To constrain the return value from `Decl` to `VarDecl`. Without this we have change all the callers to cast the return value from this function to `VarDecl`. It would be ugly.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80522/new/
https://reviews.llvm.org/D80522
More information about the cfe-commits
mailing list