diff options
| author | Balazs Benics <benicsbalazs@gmail.com> | 2020-11-06 16:51:34 +0100 |
|---|---|---|
| committer | Balazs Benics <benicsbalazs@gmail.com> | 2020-11-06 16:51:34 +0100 |
| commit | aa0dc1c3b8628bed8c8b91a75258d13eefa88dfe (patch) | |
| tree | e14bd3ab8d8ef5257a990d22237e811627473156 /clang/lib/CodeGen/CodeGenFunction.cpp | |
| parent | 7efe07a12b6967bc358678adf75dffd238a00250 (diff) | |
| download | llvm-aa0dc1c3b8628bed8c8b91a75258d13eefa88dfe.zip llvm-aa0dc1c3b8628bed8c8b91a75258d13eefa88dfe.tar.gz llvm-aa0dc1c3b8628bed8c8b91a75258d13eefa88dfe.tar.bz2 | |
[analyzer][NFCi] Mark CallEvent::getOriginExpr virtual, some cleanup
Previously we just shadowed the original implementation with a virtual
declaration - which is really bugprone in a long run.
This patch marks `CallEvent::getOriginExpr` virtual to let subclasses
override it's behavior.
At the same time, I checked all virtual functions of this class hierarchy
to make sure we don't suffer from this elsewhere.
Removes redundant declarations of `virtual` if `override` is already present.
In theory, this patch is a functional change, but no tests were broken.
I suspect that there were no meaningful changes in behavior in the
subclasses compared to the shadowed `CallEvent::getOriginExpr`.
That being said, I had a hard time coming up with unit-tests covering this.
Motivation: https://reviews.llvm.org/D74735#2370909
Reviewed By: NoQ
Differential Revision: https://reviews.llvm.org/D90754
Diffstat (limited to 'clang/lib/CodeGen/CodeGenFunction.cpp')
0 files changed, 0 insertions, 0 deletions