https://github.com/capfredf updated https://github.com/llvm/llvm-project/pull/67349
>From 4b371d40d74d6297bdb5ecbe2eae0573e20d0569 Mon Sep 17 00:00:00 2001 From: Fred Fu <moons...@gmail.com> Date: Tue, 29 Aug 2023 11:56:59 -0400 Subject: [PATCH 1/8] [ClangRepl] Type Directed Code Completion Differential Revision: https://reviews.llvm.org/D159128 --- .../clang/Interpreter/CodeCompletion.h | 11 +- clang/lib/Interpreter/CodeCompletion.cpp | 147 +++++++++++-- clang/lib/Sema/SemaLookup.cpp | 4 + clang/tools/clang-repl/ClangRepl.cpp | 25 +-- .../Interpreter/CodeCompletionTest.cpp | 195 +++++++++++++++++- 5 files changed, 334 insertions(+), 48 deletions(-) diff --git a/clang/include/clang/Interpreter/CodeCompletion.h b/clang/include/clang/Interpreter/CodeCompletion.h index 9adcdf0dc3afac6..83f665b7a2db944 100644 --- a/clang/include/clang/Interpreter/CodeCompletion.h +++ b/clang/include/clang/Interpreter/CodeCompletion.h @@ -23,8 +23,13 @@ namespace clang { class CodeCompletionResult; class CompilerInstance; -void codeComplete(CompilerInstance *InterpCI, llvm::StringRef Content, - unsigned Line, unsigned Col, const CompilerInstance *ParentCI, - std::vector<std::string> &CCResults); +struct ReplCodeCompletion { + ReplCodeCompletion() = default; + std::string Prefix; + void codeComplete(CompilerInstance *InterpCI, llvm::StringRef Content, + unsigned Line, unsigned Col, + const CompilerInstance *ParentCI, + std::vector<std::string> &CCResults); +}; } // namespace clang #endif diff --git a/clang/lib/Interpreter/CodeCompletion.cpp b/clang/lib/Interpreter/CodeCompletion.cpp index c40e11b9d1ece0a..9c812c28f677726 100644 --- a/clang/lib/Interpreter/CodeCompletion.cpp +++ b/clang/lib/Interpreter/CodeCompletion.cpp @@ -23,6 +23,9 @@ #include "clang/Sema/CodeCompleteConsumer.h" #include "clang/Sema/CodeCompleteOptions.h" #include "clang/Sema/Sema.h" +#include "llvm/Support/Debug.h" +#define DEBUG_TYPE "REPLCC" + namespace clang { @@ -37,12 +40,22 @@ clang::CodeCompleteOptions getClangCompleteOpts() { return Opts; } +class CodeCompletionSubContext { +public: + virtual ~CodeCompletionSubContext(){}; + virtual void + HandleCodeCompleteResults(class Sema &S, CodeCompletionResult *InResults, + unsigned NumResults, + std::vector<std::string> &Results) = 0; +}; + class ReplCompletionConsumer : public CodeCompleteConsumer { public: - ReplCompletionConsumer(std::vector<std::string> &Results) + ReplCompletionConsumer(std::vector<std::string> &Results, ReplCodeCompletion& CC) : CodeCompleteConsumer(getClangCompleteOpts()), CCAllocator(std::make_shared<GlobalCodeCompletionAllocator>()), - CCTUInfo(CCAllocator), Results(Results){}; + CCTUInfo(CCAllocator), Results(Results), CC(CC) { + } void ProcessCodeCompleteResults(class Sema &S, CodeCompletionContext Context, CodeCompletionResult *InResults, @@ -56,26 +69,90 @@ class ReplCompletionConsumer : public CodeCompleteConsumer { std::shared_ptr<GlobalCodeCompletionAllocator> CCAllocator; CodeCompletionTUInfo CCTUInfo; std::vector<std::string> &Results; + ReplCodeCompletion& CC; }; void ReplCompletionConsumer::ProcessCodeCompleteResults( class Sema &S, CodeCompletionContext Context, CodeCompletionResult *InResults, unsigned NumResults) { - for (unsigned I = 0; I < NumResults; ++I) { - auto &Result = InResults[I]; - switch (Result.Kind) { - case CodeCompletionResult::RK_Declaration: - if (auto *ID = Result.Declaration->getIdentifier()) { - Results.push_back(ID->getName().str()); + + // auto& a = S.Context.Idents.get("f1"); + // LLVM_DEBUG(llvm::dbgs() << "\n FFFFF111111 : " << a.getName() << "\n" ); + auto Prefix = S.getPreprocessor().getCodeCompletionFilter(); + CC.Prefix = Prefix; + switch (Context.getKind()) { + case CodeCompletionContext::CCC_DotMemberAccess: { + // FIXME: check BaseType is dependent, or from a typo + auto BaseType = Context.getBaseType(); + LLVM_DEBUG(llvm::dbgs() << "BaseType : " << BaseType.getAsString() << "\n" ); + for (unsigned I = 0; I < NumResults; ++I) { + auto &Result = InResults[I]; + switch (Result.Kind) { + case CodeCompletionResult::RK_Declaration: + if (auto *ID = Result.Declaration->getIdentifier()) { + if (const auto *Fun = llvm::dyn_cast<CXXMethodDecl>(Result.Declaration)) { + if (Fun->getParent()->getCanonicalDecl() == BaseType->getAsCXXRecordDecl()->getCanonicalDecl()) { + LLVM_DEBUG(llvm::dbgs() << "[In HandleCodeCompleteDOT] Name : " << ID->getName() << "\n"); + Results.push_back(ID->getName().str()); + } + } + } + break; + default: + break; } - break; - case CodeCompletionResult::RK_Keyword: - Results.push_back(Result.Keyword); - break; - default: - break; } + break; } + default: + auto PreferredType = Context.getPreferredType(); + for (unsigned I = 0; I < NumResults; I++) { + auto &Result = InResults[I]; + switch (Result.Kind) { + case CodeCompletionResult::RK_Declaration: + if (Result.Hidden) { + break; + } + if (!Result.Declaration->getDeclName().isIdentifier() || !Result.Declaration->getName().startswith(Prefix)) { + break; + } + if (!PreferredType.isNull()) { + if (auto* VD = dyn_cast<VarDecl>(Result.Declaration)) { + auto ArgumentType = VD->getType(); + if (PreferredType->isReferenceType()) { + QualType RT = PreferredType->castAs<ReferenceType>()->getPointeeType(); + Sema::ReferenceConversions RefConv; + Sema::ReferenceCompareResult RefRelationship = + S.CompareReferenceRelationship(SourceLocation(), RT, ArgumentType, + &RefConv); + switch (RefRelationship) { + case Sema::Ref_Compatible: + case Sema::Ref_Related: + Results.push_back(VD->getName().str()); + break; + case Sema::Ref_Incompatible: + break; + } + } else if (S.Context.hasSameType(ArgumentType, PreferredType)) { + Results.push_back(VD->getName().str()); + } + } + } else + Results.push_back(Result.Declaration->getName().str()); + break; + case CodeCompletionResult::RK_Keyword: + // add keyword to the completion results only if we are in a type-aware situation. + if (!Context.getBaseType().isNull() || !PreferredType.isNull()) + break; + if (StringRef(Result.Keyword).startswith(Prefix)) + Results.push_back(Result.Keyword); + break; + default: + break; + } + } + } + std::sort(Results.begin(), Results.end()); } class IncrementalSyntaxOnlyAction : public SyntaxOnlyAction { @@ -101,8 +178,8 @@ class ExternalSource : public clang::ExternalASTSource { ASTContext &ParentASTCtxt, FileManager &ParentFM); bool FindExternalVisibleDeclsByName(const DeclContext *DC, DeclarationName Name) override; - void - completeVisibleDeclsMap(const clang::DeclContext *childDeclContext) override; + // void CompleteType(TagDecl *Tag) override; + void completeVisibleDeclsMap(const clang::DeclContext *childDeclContext) override; }; // This method is intended to set up `ExternalASTSource` to the running @@ -134,6 +211,7 @@ ExternalSource::ExternalSource(ASTContext &ChildASTCtxt, FileManager &ChildFM, bool ExternalSource::FindExternalVisibleDeclsByName(const DeclContext *DC, DeclarationName Name) { + IdentifierTable &ParentIdTable = ParentASTCtxt.Idents; auto ParentDeclName = @@ -148,6 +226,10 @@ bool ExternalSource::FindExternalVisibleDeclsByName(const DeclContext *DC, return false; } +// void ExternalSource::CompleteType(TagDecl *Tag) { +// LLVM_DEBUG(llvm::dbgs() << "\n CompleteType " << Tag->getName() << ":\n"); +// } + void ExternalSource::completeVisibleDeclsMap( const DeclContext *ChildDeclContext) { assert(ChildDeclContext && ChildDeclContext == ChildTUDeclCtxt && @@ -162,12 +244,36 @@ void ExternalSource::completeVisibleDeclsMap( if (NamedDecl *Decl = llvm::dyn_cast<NamedDecl>(IDeclContext)) { if (auto DeclOrErr = Importer->Import(Decl)) { if (NamedDecl *importedNamedDecl = - llvm::dyn_cast<NamedDecl>(*DeclOrErr)) { + llvm::dyn_cast<NamedDecl>(*DeclOrErr)) { + // LLVM_DEBUG(llvm::dbgs() << "\nImporting : " << importedNamedDecl->getName() << "\n"); SetExternalVisibleDeclsForName(ChildDeclContext, importedNamedDecl->getDeclName(), importedNamedDecl); - } + if (auto *Record = llvm::dyn_cast<CXXRecordDecl>(importedNamedDecl)) { + if (auto Err = Importer->ImportDefinition(Decl)) + consumeError(std::move(Err)); + // LLVM_DEBUG(llvm::dbgs() << "\nHello :\n"); + // Record->getTypeForDecl()->dump(); + Record->setHasLoadedFieldsFromExternalStorage(true); + // auto ToOrErr = Importer->Import(->getTypeForDecl()); + // if (!ToOrErr) { + // consumeError(std::move(ToOrErr.takeError())); + // } + LLVM_DEBUG(llvm::dbgs() + << "\nCXXRecrod : " << Record->getName() + << " size(methods): " << std::distance(Record->method_begin(), Record->method_end()) + << " has def?: " << Record->hasDefinition() + << " # (methods): " << std::distance(Record->getDefinition()->method_begin(), Record->getDefinition()->method_end()) + << "\n"); + for (auto *Meth : Record->methods()) { + SetExternalVisibleDeclsForName(ChildDeclContext, Meth->getDeclName(), Meth); + // if (Meth->getDeclName().isIdentifier()) { + // LLVM_DEBUG(llvm::dbgs() << "CXXRecrod Method: " << Meth->getName() << "\n"); + // } + } + } + } } else { llvm::consumeError(DeclOrErr.takeError()); } @@ -177,11 +283,12 @@ void ExternalSource::completeVisibleDeclsMap( } } -void codeComplete(CompilerInstance *InterpCI, llvm::StringRef Content, + +void ReplCodeCompletion::codeComplete(CompilerInstance *InterpCI, llvm::StringRef Content, unsigned Line, unsigned Col, const CompilerInstance *ParentCI, std::vector<std::string> &CCResults) { auto DiagOpts = DiagnosticOptions(); - auto consumer = ReplCompletionConsumer(CCResults); + auto consumer = ReplCompletionConsumer(CCResults, *this); auto diag = InterpCI->getDiagnosticsPtr(); std::unique_ptr<ASTUnit> AU(ASTUnit::LoadFromCompilerInvocationAction( diff --git a/clang/lib/Sema/SemaLookup.cpp b/clang/lib/Sema/SemaLookup.cpp index 324a57095560b79..f76871b84ac31b9 100644 --- a/clang/lib/Sema/SemaLookup.cpp +++ b/clang/lib/Sema/SemaLookup.cpp @@ -51,6 +51,8 @@ #include <vector> #include "OpenCLBuiltins.inc" +#include "llvm/Support/Debug.h" +#define DEBUG_TYPE "REPLCC" using namespace clang; using namespace sema; @@ -2748,6 +2750,8 @@ bool Sema::LookupParsedName(LookupResult &R, Scope *S, CXXScopeSpec *SS, return false; } + Context.getTranslationUnitDecl()->lookups(); + // Perform unqualified name lookup starting in the given scope. return LookupName(R, S, AllowBuiltinCreation); } diff --git a/clang/tools/clang-repl/ClangRepl.cpp b/clang/tools/clang-repl/ClangRepl.cpp index 51741fd1a27ef4a..a428b4b1d7045b9 100644 --- a/clang/tools/clang-repl/ClangRepl.cpp +++ b/clang/tools/clang-repl/ClangRepl.cpp @@ -15,6 +15,8 @@ #include "clang/Frontend/FrontendDiagnostic.h" #include "clang/Interpreter/CodeCompletion.h" #include "clang/Interpreter/Interpreter.h" +#include "clang/Lex/Preprocessor.h" +#include "clang/Sema/Sema.h" #include "llvm/ExecutionEngine/Orc/LLJIT.h" #include "llvm/LineEditor/LineEditor.h" @@ -115,22 +117,15 @@ ReplListCompleter::operator()(llvm::StringRef Buffer, size_t Pos, return {}; } - - codeComplete( - const_cast<clang::CompilerInstance *>((*Interp)->getCompilerInstance()), - Buffer, Lines, Pos + 1, MainInterp.getCompilerInstance(), Results); - - size_t space_pos = Buffer.rfind(" "); - llvm::StringRef Prefix; - if (space_pos == llvm::StringRef::npos) { - Prefix = Buffer; - } else { - Prefix = Buffer.substr(space_pos + 1); - } - + auto *MainCI = + const_cast<clang::CompilerInstance *>((*Interp)->getCompilerInstance()); + auto CC = clang::ReplCodeCompletion(); + CC.codeComplete(MainCI, Buffer, Lines, Pos + 1, + MainInterp.getCompilerInstance(), Results); for (auto c : Results) { - if (c.find(Prefix) == 0) - Comps.push_back(llvm::LineEditor::Completion(c.substr(Prefix.size()), c)); + if (c.find(CC.Prefix) == 0) + Comps.push_back( + llvm::LineEditor::Completion(c.substr(CC.Prefix.size()), c)); } return Comps; } diff --git a/clang/unittests/Interpreter/CodeCompletionTest.cpp b/clang/unittests/Interpreter/CodeCompletionTest.cpp index 8f5f3545029d087..b1b4e06a660fd06 100644 --- a/clang/unittests/Interpreter/CodeCompletionTest.cpp +++ b/clang/unittests/Interpreter/CodeCompletionTest.cpp @@ -1,7 +1,9 @@ #include "clang/Interpreter/CodeCompletion.h" #include "clang/Frontend/CompilerInstance.h" #include "clang/Interpreter/Interpreter.h" +#include "clang/Lex/Preprocessor.h" #include "clang/Sema/CodeCompleteConsumer.h" +#include "clang/Sema/Sema.h" #include "llvm/LineEditor/LineEditor.h" #include "llvm/Support/Error.h" #include "llvm/Support/raw_ostream.h" @@ -19,7 +21,7 @@ static std::unique_ptr<Interpreter> createInterpreter() { } static std::vector<std::string> runComp(clang::Interpreter &MainInterp, - llvm::StringRef Prefix, + llvm::StringRef Input, llvm::Error &ErrR) { auto CI = CB.CreateCpp(); if (auto Err = CI.takeError()) { @@ -37,16 +39,15 @@ static std::vector<std::string> runComp(clang::Interpreter &MainInterp, std::vector<std::string> Results; std::vector<std::string> Comps; - - codeComplete( - const_cast<clang::CompilerInstance *>((*Interp)->getCompilerInstance()), - Prefix, /* Lines */ 1, Prefix.size(), MainInterp.getCompilerInstance(), - Results); + auto *MainCI = + const_cast<clang::CompilerInstance *>((*Interp)->getCompilerInstance()); + auto CC = ReplCodeCompletion(); + CC.codeComplete(MainCI, Input, /* Lines */ 1, Input.size() + 1, + MainInterp.getCompilerInstance(), Results); for (auto Res : Results) - if (Res.find(Prefix) == 0) + if (Res.find(CC.Prefix) == 0) Comps.push_back(Res); - return Comps; } @@ -62,8 +63,9 @@ TEST(CodeCompletionTest, Sanity) { } auto Err = llvm::Error::success(); auto comps = runComp(*Interp, "f", Err); - EXPECT_EQ((size_t)2, comps.size()); // foo and float - EXPECT_EQ(comps[0], std::string("foo")); + EXPECT_EQ((size_t)2, comps.size()); // float and foo + EXPECT_EQ(comps[0], std::string("float")); + EXPECT_EQ(comps[1], std::string("foo")); EXPECT_EQ((bool)Err, false); } @@ -110,4 +112,177 @@ TEST(CodeCompletionTest, CompFunDeclsNoError) { EXPECT_EQ((bool)Err, false); } +TEST(CodeCompletionTest, TypedDirected) { + auto Interp = createInterpreter(); + if (auto R = Interp->ParseAndExecute("int application = 12;")) { + consumeError(std::move(R)); + return; + } + if (auto R = Interp->ParseAndExecute("char apple = '2';")) { + consumeError(std::move(R)); + return; + } + if (auto R = Interp->ParseAndExecute("void add(int &SomeInt){}")) { + consumeError(std::move(R)); + return; + } + { + auto Err = llvm::Error::success(); + auto comps = runComp(*Interp, std::string("add("), Err); + EXPECT_EQ((size_t)1, comps.size()); + EXPECT_EQ((bool)Err, false); + } + + if (auto R = Interp->ParseAndExecute("int banana = 42;")) { + consumeError(std::move(R)); + return; + } + + { + auto Err = llvm::Error::success(); + auto comps = runComp(*Interp, std::string("add("), Err); + EXPECT_EQ((size_t)2, comps.size()); + EXPECT_EQ(comps[0], "application"); + EXPECT_EQ(comps[1], "banana"); + EXPECT_EQ((bool)Err, false); + } + + { + auto Err = llvm::Error::success(); + auto comps = runComp(*Interp, std::string("add(b"), Err); + EXPECT_EQ((size_t)1, comps.size()); + EXPECT_EQ(comps[0], "banana"); + EXPECT_EQ((bool)Err, false); + } +} + +TEST(CodeCompletionTest, SanityClasses) { + auto Interp = createInterpreter(); + if (auto R = Interp->ParseAndExecute("struct Apple{};")) { + consumeError(std::move(R)); + return; + } + if (auto R = Interp->ParseAndExecute("void takeApple(Apple &a1){}")) { + consumeError(std::move(R)); + return; + } + if (auto R = Interp->ParseAndExecute("Apple a1;")) { + consumeError(std::move(R)); + return; + } + if (auto R = Interp->ParseAndExecute("void takeAppleCopy(Apple a1){}")) { + consumeError(std::move(R)); + return; + } + + { + auto Err = llvm::Error::success(); + auto comps = runComp(*Interp, "takeApple(", Err); + EXPECT_EQ((size_t)1, comps.size()); + EXPECT_EQ(comps[0], std::string("a1")); + EXPECT_EQ((bool)Err, false); + } + { + auto Err = llvm::Error::success(); + auto comps = runComp(*Interp, std::string("takeAppleCopy("), Err); + EXPECT_EQ((size_t)1, comps.size()); + EXPECT_EQ(comps[0], std::string("a1")); + EXPECT_EQ((bool)Err, false); + } +} + +TEST(CodeCompletionTest, SubClassing) { + auto Interp = createInterpreter(); + if (auto R = Interp->ParseAndExecute("struct Fruit {};")) { + consumeError(std::move(R)); + return; + } + if (auto R = Interp->ParseAndExecute("struct Apple : Fruit{};")) { + consumeError(std::move(R)); + return; + } + if (auto R = Interp->ParseAndExecute("void takeFruit(Fruit &f){}")) { + consumeError(std::move(R)); + return; + } + if (auto R = Interp->ParseAndExecute("Apple a1;")) { + consumeError(std::move(R)); + return; + } + if (auto R = Interp->ParseAndExecute("Fruit f1;")) { + consumeError(std::move(R)); + return; + } + auto Err = llvm::Error::success(); + auto comps = runComp(*Interp, std::string("takeFruit("), Err); + EXPECT_EQ((size_t)2, comps.size()); + EXPECT_EQ(comps[0], std::string("a1")); + EXPECT_EQ(comps[1], std::string("f1")); + EXPECT_EQ((bool)Err, false); +} + +TEST(CodeCompletionTest, MultipleArguments) { + auto Interp = createInterpreter(); + if (auto R = Interp->ParseAndExecute("int foo = 42;")) { + consumeError(std::move(R)); + return; + } + if (auto R = Interp->ParseAndExecute("char fowl = 'A';")) { + consumeError(std::move(R)); + return; + } + if (auto R = Interp->ParseAndExecute("void takeTwo(int &a, char b){}")) { + consumeError(std::move(R)); + return; + } + auto Err = llvm::Error::success(); + auto comps = runComp(*Interp, std::string("takeTwo(foo, "), Err); + EXPECT_EQ((size_t)1, comps.size()); + EXPECT_EQ(comps[0], std::string("fowl")); + EXPECT_EQ((bool)Err, false); +} + +TEST(CodeCompletionTest, Methods) { + auto Interp = createInterpreter(); + cantFail(Interp->ParseAndExecute( + "struct Foo{int add(int a){return 42;} int par(int b){return 42;}};")); + cantFail(Interp->ParseAndExecute("Foo f1;")); + + auto Err = llvm::Error::success(); + auto comps = runComp(*Interp, std::string("f1."), Err); + EXPECT_EQ((size_t)2, comps.size()); + EXPECT_EQ(comps[0], std::string("add")); + EXPECT_EQ(comps[1], std::string("par")); + EXPECT_EQ((bool)Err, false); +} + +TEST(CodeCompletionTest, MethodsInvocations) { + auto Interp = createInterpreter(); + cantFail(Interp->ParseAndExecute( + "struct Foo{int add(int a){return 42;} int par(int b){return 42;}};")); + cantFail(Interp->ParseAndExecute("Foo f1;")); + cantFail(Interp->ParseAndExecute("int a = 84;")); + + auto Err = llvm::Error::success(); + auto comps = runComp(*Interp, std::string("f1.add("), Err); + EXPECT_EQ((size_t)1, comps.size()); + EXPECT_EQ(comps[0], std::string("a")); + EXPECT_EQ((bool)Err, false); +} + +TEST(CodeCompletionTest, NestedInvocations) { + auto Interp = createInterpreter(); + cantFail(Interp->ParseAndExecute( + "struct Foo{int add(int a){return 42;} int par(int b){return 42;}};")); + cantFail(Interp->ParseAndExecute("Foo f1;")); + cantFail(Interp->ParseAndExecute("int a = 84;")); + cantFail(Interp->ParseAndExecute("int plus(int a, int b) { return a + b; }")); + + auto Err = llvm::Error::success(); + auto comps = runComp(*Interp, std::string("plus(42, f1.add("), Err); + EXPECT_EQ((size_t)1, comps.size()); + EXPECT_EQ(comps[0], std::string("a")); + EXPECT_EQ((bool)Err, false); +} + } // anonymous namespace >From 623290d78f497b27ad95443c45739eda79e3766d Mon Sep 17 00:00:00 2001 From: Fred Fu <moons...@gmail.com> Date: Mon, 25 Sep 2023 14:58:36 -0400 Subject: [PATCH 2/8] move code --- clang/lib/Interpreter/CodeCompletion.cpp | 112 +++++++++++++---------- clang/lib/Sema/SemaLookup.cpp | 4 - 2 files changed, 64 insertions(+), 52 deletions(-) diff --git a/clang/lib/Interpreter/CodeCompletion.cpp b/clang/lib/Interpreter/CodeCompletion.cpp index 9c812c28f677726..2e2e3b91791ab23 100644 --- a/clang/lib/Interpreter/CodeCompletion.cpp +++ b/clang/lib/Interpreter/CodeCompletion.cpp @@ -12,6 +12,7 @@ #include "clang/Interpreter/CodeCompletion.h" #include "clang/AST/ASTImporter.h" +#include "clang/AST/DeclLookups.h" #include "clang/AST/DeclarationName.h" #include "clang/AST/ExternalASTSource.h" #include "clang/Basic/IdentifierTable.h" @@ -26,7 +27,6 @@ #include "llvm/Support/Debug.h" #define DEBUG_TYPE "REPLCC" - namespace clang { const std::string CodeCompletionFileName = "input_line_[Completion]"; @@ -43,19 +43,19 @@ clang::CodeCompleteOptions getClangCompleteOpts() { class CodeCompletionSubContext { public: virtual ~CodeCompletionSubContext(){}; - virtual void - HandleCodeCompleteResults(class Sema &S, CodeCompletionResult *InResults, - unsigned NumResults, - std::vector<std::string> &Results) = 0; + virtual void HandleCodeCompleteResults(class Sema &S, + CodeCompletionResult *InResults, + unsigned NumResults, + std::vector<std::string> &Results) = 0; }; class ReplCompletionConsumer : public CodeCompleteConsumer { public: - ReplCompletionConsumer(std::vector<std::string> &Results, ReplCodeCompletion& CC) + ReplCompletionConsumer(std::vector<std::string> &Results, + ReplCodeCompletion &CC) : CodeCompleteConsumer(getClangCompleteOpts()), CCAllocator(std::make_shared<GlobalCodeCompletionAllocator>()), - CCTUInfo(CCAllocator), Results(Results), CC(CC) { - } + CCTUInfo(CCAllocator), Results(Results), CC(CC) {} void ProcessCodeCompleteResults(class Sema &S, CodeCompletionContext Context, CodeCompletionResult *InResults, @@ -69,7 +69,7 @@ class ReplCompletionConsumer : public CodeCompleteConsumer { std::shared_ptr<GlobalCodeCompletionAllocator> CCAllocator; CodeCompletionTUInfo CCTUInfo; std::vector<std::string> &Results; - ReplCodeCompletion& CC; + ReplCodeCompletion &CC; }; void ReplCompletionConsumer::ProcessCodeCompleteResults( @@ -84,15 +84,18 @@ void ReplCompletionConsumer::ProcessCodeCompleteResults( case CodeCompletionContext::CCC_DotMemberAccess: { // FIXME: check BaseType is dependent, or from a typo auto BaseType = Context.getBaseType(); - LLVM_DEBUG(llvm::dbgs() << "BaseType : " << BaseType.getAsString() << "\n" ); + LLVM_DEBUG(llvm::dbgs() << "BaseType : " << BaseType.getAsString() << "\n"); for (unsigned I = 0; I < NumResults; ++I) { auto &Result = InResults[I]; switch (Result.Kind) { case CodeCompletionResult::RK_Declaration: if (auto *ID = Result.Declaration->getIdentifier()) { - if (const auto *Fun = llvm::dyn_cast<CXXMethodDecl>(Result.Declaration)) { - if (Fun->getParent()->getCanonicalDecl() == BaseType->getAsCXXRecordDecl()->getCanonicalDecl()) { - LLVM_DEBUG(llvm::dbgs() << "[In HandleCodeCompleteDOT] Name : " << ID->getName() << "\n"); + if (const auto *Fun = + llvm::dyn_cast<CXXMethodDecl>(Result.Declaration)) { + if (Fun->getParent()->getCanonicalDecl() == + BaseType->getAsCXXRecordDecl()->getCanonicalDecl()) { + LLVM_DEBUG(llvm::dbgs() << "[In HandleCodeCompleteDOT] Name : " + << ID->getName() << "\n"); Results.push_back(ID->getName().str()); } } @@ -113,18 +116,20 @@ void ReplCompletionConsumer::ProcessCodeCompleteResults( if (Result.Hidden) { break; } - if (!Result.Declaration->getDeclName().isIdentifier() || !Result.Declaration->getName().startswith(Prefix)) { + if (!Result.Declaration->getDeclName().isIdentifier() || + !Result.Declaration->getName().startswith(Prefix)) { break; } if (!PreferredType.isNull()) { - if (auto* VD = dyn_cast<VarDecl>(Result.Declaration)) { + if (auto *VD = dyn_cast<VarDecl>(Result.Declaration)) { auto ArgumentType = VD->getType(); if (PreferredType->isReferenceType()) { - QualType RT = PreferredType->castAs<ReferenceType>()->getPointeeType(); + QualType RT = + PreferredType->castAs<ReferenceType>()->getPointeeType(); Sema::ReferenceConversions RefConv; Sema::ReferenceCompareResult RefRelationship = - S.CompareReferenceRelationship(SourceLocation(), RT, ArgumentType, - &RefConv); + S.CompareReferenceRelationship(SourceLocation(), RT, + ArgumentType, &RefConv); switch (RefRelationship) { case Sema::Ref_Compatible: case Sema::Ref_Related: @@ -141,7 +146,8 @@ void ReplCompletionConsumer::ProcessCodeCompleteResults( Results.push_back(Result.Declaration->getName().str()); break; case CodeCompletionResult::RK_Keyword: - // add keyword to the completion results only if we are in a type-aware situation. + // add keyword to the completion results only if we are in a type-aware + // situation. if (!Context.getBaseType().isNull() || !PreferredType.isNull()) break; if (StringRef(Result.Keyword).startswith(Prefix)) @@ -179,7 +185,8 @@ class ExternalSource : public clang::ExternalASTSource { bool FindExternalVisibleDeclsByName(const DeclContext *DC, DeclarationName Name) override; // void CompleteType(TagDecl *Tag) override; - void completeVisibleDeclsMap(const clang::DeclContext *childDeclContext) override; + void + completeVisibleDeclsMap(const clang::DeclContext *childDeclContext) override; }; // This method is intended to set up `ExternalASTSource` to the running @@ -195,6 +202,7 @@ void IncrementalSyntaxOnlyAction::ExecuteAction() { CI.getASTContext().getTranslationUnitDecl()->setHasExternalVisibleStorage( true); + CI.getASTContext().getTranslationUnitDecl()->lookups(); SyntaxOnlyAction::ExecuteAction(); } @@ -244,36 +252,43 @@ void ExternalSource::completeVisibleDeclsMap( if (NamedDecl *Decl = llvm::dyn_cast<NamedDecl>(IDeclContext)) { if (auto DeclOrErr = Importer->Import(Decl)) { if (NamedDecl *importedNamedDecl = - llvm::dyn_cast<NamedDecl>(*DeclOrErr)) { - // LLVM_DEBUG(llvm::dbgs() << "\nImporting : " << importedNamedDecl->getName() << "\n"); + llvm::dyn_cast<NamedDecl>(*DeclOrErr)) { + // LLVM_DEBUG(llvm::dbgs() << "\nImporting : " << + // importedNamedDecl->getName() << "\n"); SetExternalVisibleDeclsForName(ChildDeclContext, importedNamedDecl->getDeclName(), importedNamedDecl); - if (auto *Record = llvm::dyn_cast<CXXRecordDecl>(importedNamedDecl)) { - if (auto Err = Importer->ImportDefinition(Decl)) - consumeError(std::move(Err)); - // LLVM_DEBUG(llvm::dbgs() << "\nHello :\n"); - // Record->getTypeForDecl()->dump(); - Record->setHasLoadedFieldsFromExternalStorage(true); - // auto ToOrErr = Importer->Import(->getTypeForDecl()); - // if (!ToOrErr) { - // consumeError(std::move(ToOrErr.takeError())); + if (auto *Record = + llvm::dyn_cast<CXXRecordDecl>(importedNamedDecl)) { + if (auto Err = Importer->ImportDefinition(Decl)) + consumeError(std::move(Err)); + // LLVM_DEBUG(llvm::dbgs() << "\nHello :\n"); + // Record->getTypeForDecl()->dump(); + Record->setHasLoadedFieldsFromExternalStorage(true); + // auto ToOrErr = Importer->Import(->getTypeForDecl()); + // if (!ToOrErr) { + // consumeError(std::move(ToOrErr.takeError())); + // } + LLVM_DEBUG( + llvm::dbgs() + << "\nCXXRecrod : " << Record->getName() << " size(methods): " + << std::distance(Record->method_begin(), Record->method_end()) + << " has def?: " << Record->hasDefinition() + << " # (methods): " + << std::distance(Record->getDefinition()->method_begin(), + Record->getDefinition()->method_end()) + << "\n"); + for (auto *Meth : Record->methods()) { + SetExternalVisibleDeclsForName(ChildDeclContext, + Meth->getDeclName(), Meth); + // if (Meth->getDeclName().isIdentifier()) { + // LLVM_DEBUG(llvm::dbgs() << "CXXRecrod Method: " << + // Meth->getName() << "\n"); + // } - LLVM_DEBUG(llvm::dbgs() - << "\nCXXRecrod : " << Record->getName() - << " size(methods): " << std::distance(Record->method_begin(), Record->method_end()) - << " has def?: " << Record->hasDefinition() - << " # (methods): " << std::distance(Record->getDefinition()->method_begin(), Record->getDefinition()->method_end()) - << "\n"); - for (auto *Meth : Record->methods()) { - SetExternalVisibleDeclsForName(ChildDeclContext, Meth->getDeclName(), Meth); - // if (Meth->getDeclName().isIdentifier()) { - // LLVM_DEBUG(llvm::dbgs() << "CXXRecrod Method: " << Meth->getName() << "\n"); - - // } - } } } + } } else { llvm::consumeError(DeclOrErr.takeError()); } @@ -283,10 +298,11 @@ void ExternalSource::completeVisibleDeclsMap( } } - -void ReplCodeCompletion::codeComplete(CompilerInstance *InterpCI, llvm::StringRef Content, - unsigned Line, unsigned Col, const CompilerInstance *ParentCI, - std::vector<std::string> &CCResults) { +void ReplCodeCompletion::codeComplete(CompilerInstance *InterpCI, + llvm::StringRef Content, unsigned Line, + unsigned Col, + const CompilerInstance *ParentCI, + std::vector<std::string> &CCResults) { auto DiagOpts = DiagnosticOptions(); auto consumer = ReplCompletionConsumer(CCResults, *this); diff --git a/clang/lib/Sema/SemaLookup.cpp b/clang/lib/Sema/SemaLookup.cpp index f76871b84ac31b9..324a57095560b79 100644 --- a/clang/lib/Sema/SemaLookup.cpp +++ b/clang/lib/Sema/SemaLookup.cpp @@ -51,8 +51,6 @@ #include <vector> #include "OpenCLBuiltins.inc" -#include "llvm/Support/Debug.h" -#define DEBUG_TYPE "REPLCC" using namespace clang; using namespace sema; @@ -2750,8 +2748,6 @@ bool Sema::LookupParsedName(LookupResult &R, Scope *S, CXXScopeSpec *SS, return false; } - Context.getTranslationUnitDecl()->lookups(); - // Perform unqualified name lookup starting in the given scope. return LookupName(R, S, AllowBuiltinCreation); } >From 675f49a1fe0f04d14be595dcc3a3e672d84db130 Mon Sep 17 00:00:00 2001 From: Fred Fu <moons...@gmail.com> Date: Mon, 25 Sep 2023 16:13:28 -0400 Subject: [PATCH 3/8] more tests --- .../Interpreter/CodeCompletionTest.cpp | 25 +++++++++++++++++++ 1 file changed, 25 insertions(+) diff --git a/clang/unittests/Interpreter/CodeCompletionTest.cpp b/clang/unittests/Interpreter/CodeCompletionTest.cpp index b1b4e06a660fd06..fba6e0dbcd311d1 100644 --- a/clang/unittests/Interpreter/CodeCompletionTest.cpp +++ b/clang/unittests/Interpreter/CodeCompletionTest.cpp @@ -285,4 +285,29 @@ TEST(CodeCompletionTest, NestedInvocations) { EXPECT_EQ((bool)Err, false); } +TEST(CodeCompletionTest, TemplateFunctions) { + auto Interp = createInterpreter(); + cantFail(Interp->ParseAndExecute( + "template <typename T> T id(T a) { return a;} ")); + cantFail(Interp->ParseAndExecute("int apple = 84;")); + { + auto Err = llvm::Error::success(); + auto comps = runComp(*Interp, std::string("id<int>("), Err); + EXPECT_EQ((size_t)1, comps.size()); + EXPECT_EQ(comps[0], std::string("apple")); + EXPECT_EQ((bool)Err, false); + } + + cantFail(Interp->ParseAndExecute("template <typename T> T pickFirst(T a, T b) { return a;} ")); + cantFail(Interp->ParseAndExecute("char pear = '4';")); + { + auto Err = llvm::Error::success(); + auto comps = runComp(*Interp, std::string("pickFirst(apple, "), Err); + EXPECT_EQ((size_t)1, comps.size()); + EXPECT_EQ(comps[0], std::string("apple")); + EXPECT_EQ((bool)Err, false); + } +} + + } // anonymous namespace >From dd3e3e6b8eb3f9f1f740d2a9fc8a62e1964f9153 Mon Sep 17 00:00:00 2001 From: Fred Fu <moons...@gmail.com> Date: Mon, 25 Sep 2023 16:57:46 -0400 Subject: [PATCH 4/8] refactor --- clang/lib/Interpreter/CodeCompletion.cpp | 179 ++++++++++++++--------- 1 file changed, 109 insertions(+), 70 deletions(-) diff --git a/clang/lib/Interpreter/CodeCompletion.cpp b/clang/lib/Interpreter/CodeCompletion.cpp index 2e2e3b91791ab23..d30259a18517fcd 100644 --- a/clang/lib/Interpreter/CodeCompletion.cpp +++ b/clang/lib/Interpreter/CodeCompletion.cpp @@ -72,6 +72,87 @@ class ReplCompletionConsumer : public CodeCompleteConsumer { ReplCodeCompletion &CC; }; +class CompletionContextHanndler { +protected: + CodeCompletionContext CCC; + std::vector<std::string> &Results; + +public: + CompletionContextHanndler(CodeCompletionContext CCC, + std::vector<std::string> &Results) + : CCC(CCC), Results(Results) {} + virtual void handleDeclaration(const CodeCompletionResult &Result) {} + virtual void handleKeyword(const CodeCompletionResult &Result) {} + virtual void handlePattern(const CodeCompletionResult &Result) {} + virtual void handleMacro(const CodeCompletionResult &Result) {} +}; + +class DotMemberAccessHandler : public CompletionContextHanndler { +public: + DotMemberAccessHandler(CodeCompletionContext CCC, + std::vector<std::string> &Results) + : CompletionContextHanndler(CCC, Results) {} + void handleDeclaration(const CodeCompletionResult &Result) override { + if (auto *ID = Result.Declaration->getIdentifier()) { + if (const auto *Fun = llvm::dyn_cast<CXXMethodDecl>(Result.Declaration)) { + if (Fun->getParent()->getCanonicalDecl() == + CCC.getBaseType()->getAsCXXRecordDecl()->getCanonicalDecl()) { + LLVM_DEBUG(llvm::dbgs() << "[In HandleCodeCompleteDOT] Name : " + << ID->getName() << "\n"); + Results.push_back(ID->getName().str()); + } + } + } + } +}; + +class DefaultAccessHandler : public CompletionContextHanndler { +private: + Sema &S; + +public: + DefaultAccessHandler(Sema &S, CodeCompletionContext CCC, + std::vector<std::string> &Results) + : CompletionContextHanndler(CCC, Results), S(S) {} + void handleDeclaration(const CodeCompletionResult &Result) override { + auto PreferredType = CCC.getPreferredType(); + if (!PreferredType.isNull()) { + if (auto *VD = dyn_cast<VarDecl>(Result.Declaration)) { + auto ArgumentType = VD->getType(); + if (PreferredType->isReferenceType()) { + QualType RT = + PreferredType->castAs<ReferenceType>()->getPointeeType(); + Sema::ReferenceConversions RefConv; + Sema::ReferenceCompareResult RefRelationship = + S.CompareReferenceRelationship(SourceLocation(), RT, ArgumentType, + &RefConv); + switch (RefRelationship) { + case Sema::Ref_Compatible: + case Sema::Ref_Related: + Results.push_back(VD->getName().str()); + break; + case Sema::Ref_Incompatible: + break; + } + } else if (S.Context.hasSameType(ArgumentType, PreferredType)) { + Results.push_back(VD->getName().str()); + } + } + } else + Results.push_back(Result.Declaration->getName().str()); + } + + void handleKeyword(const CodeCompletionResult &Result) override { + auto Prefix = S.getPreprocessor().getCodeCompletionFilter(); + // add keyword to the completion results only if we are in a type-aware + // situation. + if (!CCC.getBaseType().isNull() || !CCC.getPreferredType().isNull()) + return; + if (StringRef(Result.Keyword).startswith(Prefix)) + Results.push_back(Result.Keyword); + } +}; + void ReplCompletionConsumer::ProcessCodeCompleteResults( class Sema &S, CodeCompletionContext Context, CodeCompletionResult *InResults, unsigned NumResults) { @@ -80,84 +161,42 @@ void ReplCompletionConsumer::ProcessCodeCompleteResults( // LLVM_DEBUG(llvm::dbgs() << "\n FFFFF111111 : " << a.getName() << "\n" ); auto Prefix = S.getPreprocessor().getCodeCompletionFilter(); CC.Prefix = Prefix; + + std::unique_ptr<CompletionContextHanndler> CCH; + switch (Context.getKind()) { - case CodeCompletionContext::CCC_DotMemberAccess: { - // FIXME: check BaseType is dependent, or from a typo - auto BaseType = Context.getBaseType(); - LLVM_DEBUG(llvm::dbgs() << "BaseType : " << BaseType.getAsString() << "\n"); - for (unsigned I = 0; I < NumResults; ++I) { - auto &Result = InResults[I]; - switch (Result.Kind) { - case CodeCompletionResult::RK_Declaration: - if (auto *ID = Result.Declaration->getIdentifier()) { - if (const auto *Fun = - llvm::dyn_cast<CXXMethodDecl>(Result.Declaration)) { - if (Fun->getParent()->getCanonicalDecl() == - BaseType->getAsCXXRecordDecl()->getCanonicalDecl()) { - LLVM_DEBUG(llvm::dbgs() << "[In HandleCodeCompleteDOT] Name : " - << ID->getName() << "\n"); - Results.push_back(ID->getName().str()); - } - } - } - break; - default: - break; - } - } + case CodeCompletionContext::CCC_DotMemberAccess: + CCH.reset(new DotMemberAccessHandler(Context, this->Results)); break; - } default: - auto PreferredType = Context.getPreferredType(); - for (unsigned I = 0; I < NumResults; I++) { - auto &Result = InResults[I]; - switch (Result.Kind) { - case CodeCompletionResult::RK_Declaration: - if (Result.Hidden) { - break; - } - if (!Result.Declaration->getDeclName().isIdentifier() || - !Result.Declaration->getName().startswith(Prefix)) { - break; - } - if (!PreferredType.isNull()) { - if (auto *VD = dyn_cast<VarDecl>(Result.Declaration)) { - auto ArgumentType = VD->getType(); - if (PreferredType->isReferenceType()) { - QualType RT = - PreferredType->castAs<ReferenceType>()->getPointeeType(); - Sema::ReferenceConversions RefConv; - Sema::ReferenceCompareResult RefRelationship = - S.CompareReferenceRelationship(SourceLocation(), RT, - ArgumentType, &RefConv); - switch (RefRelationship) { - case Sema::Ref_Compatible: - case Sema::Ref_Related: - Results.push_back(VD->getName().str()); - break; - case Sema::Ref_Incompatible: - break; - } - } else if (S.Context.hasSameType(ArgumentType, PreferredType)) { - Results.push_back(VD->getName().str()); - } - } - } else - Results.push_back(Result.Declaration->getName().str()); - break; - case CodeCompletionResult::RK_Keyword: - // add keyword to the completion results only if we are in a type-aware - // situation. - if (!Context.getBaseType().isNull() || !PreferredType.isNull()) - break; - if (StringRef(Result.Keyword).startswith(Prefix)) - Results.push_back(Result.Keyword); + CCH.reset(new DefaultAccessHandler(S, Context, this->Results)); + }; + + for (unsigned I = 0; I < NumResults; I++) { + auto &Result = InResults[I]; + switch (Result.Kind) { + case CodeCompletionResult::RK_Declaration: + if (Result.Hidden) { break; - default: + } + if (!Result.Declaration->getDeclName().isIdentifier() || + !Result.Declaration->getName().startswith(Prefix)) { break; } + CCH->handleDeclaration(Result); + break; + case CodeCompletionResult::RK_Keyword: + CCH->handleKeyword(Result); + break; + case CodeCompletionResult::RK_Macro: + CCH->handleMacro(Result); + break; + case CodeCompletionResult::RK_Pattern: + CCH->handlePattern(Result); + break; } } + std::sort(Results.begin(), Results.end()); } >From 17bbe25642381ba0db66de2b8a82fc3103c1a37d Mon Sep 17 00:00:00 2001 From: Fred Fu <moons...@gmail.com> Date: Tue, 26 Sep 2023 13:53:56 -0400 Subject: [PATCH 5/8] refactor --- clang/lib/Interpreter/CodeCompletion.cpp | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/clang/lib/Interpreter/CodeCompletion.cpp b/clang/lib/Interpreter/CodeCompletion.cpp index d30259a18517fcd..6a1c5d2d4be42f5 100644 --- a/clang/lib/Interpreter/CodeCompletion.cpp +++ b/clang/lib/Interpreter/CodeCompletion.cpp @@ -241,6 +241,15 @@ void IncrementalSyntaxOnlyAction::ExecuteAction() { CI.getASTContext().getTranslationUnitDecl()->setHasExternalVisibleStorage( true); + // Load all external decls into current context. Under the hood, it calls + // ExternalSource::completeVisibleDeclsMap, which make all decls on the redecl + // chain visible. + // + // This is crucial to code completion on dot members, since a bound variable + // before "." would be otherwise treated out-of-scope. + // + // clang-repl> Foo f1; + // clang-repl> f1.<tab> CI.getASTContext().getTranslationUnitDecl()->lookups(); SyntaxOnlyAction::ExecuteAction(); } @@ -273,9 +282,6 @@ bool ExternalSource::FindExternalVisibleDeclsByName(const DeclContext *DC, return false; } -// void ExternalSource::CompleteType(TagDecl *Tag) { -// LLVM_DEBUG(llvm::dbgs() << "\n CompleteType " << Tag->getName() << ":\n"); -// } void ExternalSource::completeVisibleDeclsMap( const DeclContext *ChildDeclContext) { @@ -292,8 +298,6 @@ void ExternalSource::completeVisibleDeclsMap( if (auto DeclOrErr = Importer->Import(Decl)) { if (NamedDecl *importedNamedDecl = llvm::dyn_cast<NamedDecl>(*DeclOrErr)) { - // LLVM_DEBUG(llvm::dbgs() << "\nImporting : " << - // importedNamedDecl->getName() << "\n"); SetExternalVisibleDeclsForName(ChildDeclContext, importedNamedDecl->getDeclName(), importedNamedDecl); >From c92959ff31890e71c98185a1cfea4153e665396c Mon Sep 17 00:00:00 2001 From: Fred Fu <moons...@gmail.com> Date: Tue, 26 Sep 2023 13:55:37 -0400 Subject: [PATCH 6/8] format --- clang/unittests/Interpreter/CodeCompletionTest.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/clang/unittests/Interpreter/CodeCompletionTest.cpp b/clang/unittests/Interpreter/CodeCompletionTest.cpp index fba6e0dbcd311d1..918b4e8167eb73c 100644 --- a/clang/unittests/Interpreter/CodeCompletionTest.cpp +++ b/clang/unittests/Interpreter/CodeCompletionTest.cpp @@ -287,8 +287,8 @@ TEST(CodeCompletionTest, NestedInvocations) { TEST(CodeCompletionTest, TemplateFunctions) { auto Interp = createInterpreter(); - cantFail(Interp->ParseAndExecute( - "template <typename T> T id(T a) { return a;} ")); + cantFail( + Interp->ParseAndExecute("template <typename T> T id(T a) { return a;} ")); cantFail(Interp->ParseAndExecute("int apple = 84;")); { auto Err = llvm::Error::success(); @@ -298,7 +298,8 @@ TEST(CodeCompletionTest, TemplateFunctions) { EXPECT_EQ((bool)Err, false); } - cantFail(Interp->ParseAndExecute("template <typename T> T pickFirst(T a, T b) { return a;} ")); + cantFail(Interp->ParseAndExecute( + "template <typename T> T pickFirst(T a, T b) { return a;} ")); cantFail(Interp->ParseAndExecute("char pear = '4';")); { auto Err = llvm::Error::success(); @@ -309,5 +310,4 @@ TEST(CodeCompletionTest, TemplateFunctions) { } } - } // anonymous namespace >From 0d04c376ec38619c672733206c059e4ed3993372 Mon Sep 17 00:00:00 2001 From: Fred Fu <moons...@gmail.com> Date: Thu, 26 Oct 2023 12:42:27 -0400 Subject: [PATCH 7/8] format --- clang/lib/Interpreter/CodeCompletion.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/clang/lib/Interpreter/CodeCompletion.cpp b/clang/lib/Interpreter/CodeCompletion.cpp index 6a1c5d2d4be42f5..0eb493d1b894b39 100644 --- a/clang/lib/Interpreter/CodeCompletion.cpp +++ b/clang/lib/Interpreter/CodeCompletion.cpp @@ -282,7 +282,6 @@ bool ExternalSource::FindExternalVisibleDeclsByName(const DeclContext *DC, return false; } - void ExternalSource::completeVisibleDeclsMap( const DeclContext *ChildDeclContext) { assert(ChildDeclContext && ChildDeclContext == ChildTUDeclCtxt && >From 736ea3aa4455daf11803806274e18e8eaa7d7fd9 Mon Sep 17 00:00:00 2001 From: Fred Fu <moons...@gmail.com> Date: Thu, 26 Oct 2023 12:42:55 -0400 Subject: [PATCH 8/8] format --- clang/tools/clang-repl/ClangRepl.cpp | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/clang/tools/clang-repl/ClangRepl.cpp b/clang/tools/clang-repl/ClangRepl.cpp index a428b4b1d7045b9..5c04be84a621fd3 100644 --- a/clang/tools/clang-repl/ClangRepl.cpp +++ b/clang/tools/clang-repl/ClangRepl.cpp @@ -147,9 +147,7 @@ int main(int argc, const char **argv) { llvm::InitializeAllAsmPrinters(); if (OptHostSupportsJit) { - auto J = llvm::orc::LLJITBuilder() - .setEnableDebuggerSupport(true) - .create(); + auto J = llvm::orc::LLJITBuilder().setEnableDebuggerSupport(true).create(); if (J) llvm::outs() << "true\n"; else { _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
