mgehre abandoned this revision.
mgehre added a comment.
The discussions on the bug did not provide further insight (until now).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66365/new/
https://reviews.llvm.org/D66365
mgehre added a comment.
I struggle myself to fully understand the motivation behind the bug report.
This is certainly not code that I would write.
So I don't really care about that kind of code and I didn't want to be in the
way of other people's way of writing C++.
Alternatively, we could ask
aaron.ballman added a comment.
I'd like some help in understanding the motivation for this change. The bug
report says `But the intention behind this is clearly that there is a member
function A::foo which derived classes overwrite.` and I'm not certain that's so
clear. It's been my experience
mgehre created this revision.
mgehre added reviewers: aaron.ballman, Eugene.Zelenko.
Herald added a subscriber: xazax.hun.
Herald added a project: clang.
Fixes bug https://bugs.llvm.org/show_bug.cgi?id=43002
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D66365
Files:
