On Sat, Sep 26, 2015 at 8:44 PM, Craig Topper via cfe-commits < cfe-commits@lists.llvm.org> wrote:
> Author: ctopper > Date: Sat Sep 26 22:44:08 2015 > New Revision: 248678 > > URL: http://llvm.org/viewvc/llvm-project?rev=248678&view=rev > Log: > Use llvm::makeArrayRef. NFC. > > Modified: > cfe/trunk/include/clang/AST/Type.h > cfe/trunk/include/clang/Analysis/Analyses/ThreadSafetyTIL.h > cfe/trunk/lib/AST/ASTImporter.cpp > cfe/trunk/lib/CodeGen/CodeGenModule.cpp > cfe/trunk/lib/Serialization/ASTWriterDecl.cpp > > Modified: cfe/trunk/include/clang/AST/Type.h > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/Type.h?rev=248678&r1=248677&r2=248678&view=diff > > ============================================================================== > --- cfe/trunk/include/clang/AST/Type.h (original) > +++ cfe/trunk/include/clang/AST/Type.h Sat Sep 26 22:44:08 2015 > @@ -4635,7 +4635,7 @@ public: > /// Retrieve the type arguments of this object type as they were > /// written. > ArrayRef<QualType> getTypeArgsAsWritten() const { > - return ArrayRef<QualType>(getTypeArgStorage(), > + return llvm::makeArrayRef(getTypeArgStorage(), > ObjCObjectTypeBits.NumTypeArgs); > } > > > Modified: cfe/trunk/include/clang/Analysis/Analyses/ThreadSafetyTIL.h > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Analysis/Analyses/ThreadSafetyTIL.h?rev=248678&r1=248677&r2=248678&view=diff > > ============================================================================== > --- cfe/trunk/include/clang/Analysis/Analyses/ThreadSafetyTIL.h (original) > +++ cfe/trunk/include/clang/Analysis/Analyses/ThreadSafetyTIL.h Sat Sep 26 > 22:44:08 2015 > @@ -1395,7 +1395,7 @@ public: > > /// Return the list of basic blocks that this terminator can branch to. > ArrayRef<BasicBlock*> successors() { > - return ArrayRef<BasicBlock*>(&TargetBlock, 1); > + return llvm::makeArrayRef(TargetBlock); > I think this one might work even without makeArrayRef? (there's an implicit ctor in ArrayRef<T> from T) > } > > template <class V> > @@ -1445,7 +1445,7 @@ public: > > /// Return the list of basic blocks that this terminator can branch to. > ArrayRef<BasicBlock*> successors() { > - return ArrayRef<BasicBlock*>(Branches, 2); > + return llvm::makeArrayRef(Branches); > } > > template <class V> > > Modified: cfe/trunk/lib/AST/ASTImporter.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/ASTImporter.cpp?rev=248678&r1=248677&r2=248678&view=diff > > ============================================================================== > --- cfe/trunk/lib/AST/ASTImporter.cpp (original) > +++ cfe/trunk/lib/AST/ASTImporter.cpp Sat Sep 26 22:44:08 2015 > @@ -5329,7 +5329,7 @@ Expr *ASTNodeImporter::VisitCallExpr(Cal > > return new (Importer.getToContext()) > CallExpr(Importer.getToContext(), ToCallee, > - ArrayRef<Expr*>(ToArgs_Copied, NumArgs), T, > E->getValueKind(), > + llvm::makeArrayRef(ToArgs_Copied, NumArgs), T, > E->getValueKind(), > Importer.Import(E->getRParenLoc())); > } > > > Modified: cfe/trunk/lib/CodeGen/CodeGenModule.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CodeGenModule.cpp?rev=248678&r1=248677&r2=248678&view=diff > > ============================================================================== > --- cfe/trunk/lib/CodeGen/CodeGenModule.cpp (original) > +++ cfe/trunk/lib/CodeGen/CodeGenModule.cpp Sat Sep 26 22:44:08 2015 > @@ -2825,7 +2825,7 @@ CodeGenModule::GetAddrOfConstantCFString > // String pointer. > llvm::Constant *C = nullptr; > if (isUTF16) { > - ArrayRef<uint16_t> Arr = llvm::makeArrayRef<uint16_t>( > + auto Arr = llvm::makeArrayRef( > reinterpret_cast<uint16_t *>(const_cast<char > *>(Entry.first().data())), > Entry.first().size() / 2); > C = llvm::ConstantDataArray::get(VMContext, Arr); > > Modified: cfe/trunk/lib/Serialization/ASTWriterDecl.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Serialization/ASTWriterDecl.cpp?rev=248678&r1=248677&r2=248678&view=diff > > ============================================================================== > --- cfe/trunk/lib/Serialization/ASTWriterDecl.cpp (original) > +++ cfe/trunk/lib/Serialization/ASTWriterDecl.cpp Sat Sep 26 22:44:08 2015 > @@ -208,7 +208,7 @@ namespace clang { > auto &&PartialSpecializations = getPartialSpecializations(Common); > ArrayRef<DeclID> LazySpecializations; > if (auto *LS = Common->LazySpecializations) > - LazySpecializations = ArrayRef<DeclID>(LS + 1, LS + 1 + LS[0]); > + LazySpecializations = llvm::makeArrayRef(LS + 1, LS + 1 + LS[0]); > Maybe you could use LS[0] as the second arg (for a ptr+length rather than begin+end construction)? > > // Add a slot to the record for the number of specializations. > unsigned I = Record.size(); > > > _______________________________________________ > cfe-commits mailing list > cfe-commits@lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
