forked from OSchip/llvm-project
Nitpicky refactoring -- use of nullptr and auto, made a bit more const-correct. No functional changes intended.
llvm-svn: 206491
This commit is contained in:
parent
11a6082e33
commit
7a2fb5f436
|
@ -1923,43 +1923,43 @@ static bool mergeAlignedAttrs(Sema &S, NamedDecl *New, Decl *Old) {
|
|||
return AnyAdded;
|
||||
}
|
||||
|
||||
static bool mergeDeclAttribute(Sema &S, NamedDecl *D, InheritableAttr *Attr,
|
||||
bool Override) {
|
||||
InheritableAttr *NewAttr = NULL;
|
||||
static bool mergeDeclAttribute(Sema &S, NamedDecl *D,
|
||||
const InheritableAttr *Attr, bool Override) {
|
||||
InheritableAttr *NewAttr = nullptr;
|
||||
unsigned AttrSpellingListIndex = Attr->getSpellingListIndex();
|
||||
if (AvailabilityAttr *AA = dyn_cast<AvailabilityAttr>(Attr))
|
||||
if (const auto *AA = dyn_cast<AvailabilityAttr>(Attr))
|
||||
NewAttr = S.mergeAvailabilityAttr(D, AA->getRange(), AA->getPlatform(),
|
||||
AA->getIntroduced(), AA->getDeprecated(),
|
||||
AA->getObsoleted(), AA->getUnavailable(),
|
||||
AA->getMessage(), Override,
|
||||
AttrSpellingListIndex);
|
||||
else if (VisibilityAttr *VA = dyn_cast<VisibilityAttr>(Attr))
|
||||
else if (const auto *VA = dyn_cast<VisibilityAttr>(Attr))
|
||||
NewAttr = S.mergeVisibilityAttr(D, VA->getRange(), VA->getVisibility(),
|
||||
AttrSpellingListIndex);
|
||||
else if (TypeVisibilityAttr *VA = dyn_cast<TypeVisibilityAttr>(Attr))
|
||||
else if (const auto *VA = dyn_cast<TypeVisibilityAttr>(Attr))
|
||||
NewAttr = S.mergeTypeVisibilityAttr(D, VA->getRange(), VA->getVisibility(),
|
||||
AttrSpellingListIndex);
|
||||
else if (DLLImportAttr *ImportA = dyn_cast<DLLImportAttr>(Attr))
|
||||
else if (const auto *ImportA = dyn_cast<DLLImportAttr>(Attr))
|
||||
NewAttr = S.mergeDLLImportAttr(D, ImportA->getRange(),
|
||||
AttrSpellingListIndex);
|
||||
else if (DLLExportAttr *ExportA = dyn_cast<DLLExportAttr>(Attr))
|
||||
else if (const auto *ExportA = dyn_cast<DLLExportAttr>(Attr))
|
||||
NewAttr = S.mergeDLLExportAttr(D, ExportA->getRange(),
|
||||
AttrSpellingListIndex);
|
||||
else if (FormatAttr *FA = dyn_cast<FormatAttr>(Attr))
|
||||
else if (const auto *FA = dyn_cast<FormatAttr>(Attr))
|
||||
NewAttr = S.mergeFormatAttr(D, FA->getRange(), FA->getType(),
|
||||
FA->getFormatIdx(), FA->getFirstArg(),
|
||||
AttrSpellingListIndex);
|
||||
else if (SectionAttr *SA = dyn_cast<SectionAttr>(Attr))
|
||||
else if (const auto *SA = dyn_cast<SectionAttr>(Attr))
|
||||
NewAttr = S.mergeSectionAttr(D, SA->getRange(), SA->getName(),
|
||||
AttrSpellingListIndex);
|
||||
else if (MSInheritanceAttr *IA = dyn_cast<MSInheritanceAttr>(Attr))
|
||||
else if (const auto *IA = dyn_cast<MSInheritanceAttr>(Attr))
|
||||
NewAttr = S.mergeMSInheritanceAttr(D, IA->getRange(), IA->getBestCase(),
|
||||
AttrSpellingListIndex,
|
||||
IA->getSemanticSpelling());
|
||||
else if (isa<AlignedAttr>(Attr))
|
||||
// AlignedAttrs are handled separately, because we need to handle all
|
||||
// such attributes on a declaration at the same time.
|
||||
NewAttr = 0;
|
||||
NewAttr = nullptr;
|
||||
else if (Attr->duplicatesAllowed() || !DeclHasAttr(D, Attr))
|
||||
NewAttr = cast<InheritableAttr>(Attr->clone(S.Context));
|
||||
|
||||
|
|
Loading…
Reference in New Issue