[PATCH] D66179: [LifetimeAnalysis] Support more STL idioms (template forward declaration and DependentNameType)
Matthias Gehre via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 21 15:07:42 PDT 2019
mgehre marked 3 inline comments as done.
mgehre added inline comments.
================
Comment at: clang/lib/Sema/SemaDeclAttr.cpp:4596
+ for (Decl *Redecl : D->redecls()) {
+ Redecl->addAttr(::new (S.Context)
+ OwnerAttr(AL.getRange(), S.Context, DerefTypeLoc,
----------------
gribozavr wrote:
> Add it only if it does not already exist?
If a previous decl had that Attribute, then we would have inherited it, and the check above for an existing attribute would have led to an early return.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66179/new/
https://reviews.llvm.org/D66179
More information about the cfe-commits
mailing list