[cling] Completely remove setting CodeCompletionOffset

This is not needed anymore as cling follows the same autocomplete
logic as `clang-repl`

Author: devajithvs

interpreter/cling/include/cling/Interpreter/CompilationOptions.h

@@ -70,12 +70,6 @@ namespace cling {
     ///\brief Optimization level.
     unsigned OptLevel : 2;
 
-    ///\brief Offset into the input line to enable the setting of the
-    /// code completion point.
-    /// -1 diasables code completion.
-    ///
-    int CodeCompletionOffset = -1;
-
     CompilationOptions() {
       DeclarationExtraction = 0;
       EnableShadowing = 0;
@@ -91,35 +85,29 @@ namespace cling {
     }
 
     bool operator==(CompilationOptions Other) const {
-      return
-        DeclarationExtraction == Other.DeclarationExtraction &&
-        EnableShadowing       == Other.EnableShadowing &&
-        ValuePrinting         == Other.ValuePrinting &&
-        ResultEvaluation      == Other.ResultEvaluation &&
-        DynamicScoping        == Other.DynamicScoping &&
-        Debug                 == Other.Debug &&
-        CodeGeneration        == Other.CodeGeneration &&
-        CodeGenerationForModule == Other.CodeGenerationForModule &&
-        IgnorePromptDiags     == Other.IgnorePromptDiags &&
-        CheckPointerValidity  == Other.CheckPointerValidity &&
-        OptLevel              == Other.OptLevel &&
-        CodeCompletionOffset  == Other.CodeCompletionOffset;
+      return DeclarationExtraction == Other.DeclarationExtraction &&
+             EnableShadowing == Other.EnableShadowing &&
+             ValuePrinting == Other.ValuePrinting &&
+             ResultEvaluation == Other.ResultEvaluation &&
+             DynamicScoping == Other.DynamicScoping && Debug == Other.Debug &&
+             CodeGeneration == Other.CodeGeneration &&
+             CodeGenerationForModule == Other.CodeGenerationForModule &&
+             IgnorePromptDiags == Other.IgnorePromptDiags &&
+             CheckPointerValidity == Other.CheckPointerValidity &&
+             OptLevel == Other.OptLevel;
     }
 
     bool operator!=(CompilationOptions Other) const {
-      return
-        DeclarationExtraction != Other.DeclarationExtraction ||
-        EnableShadowing       != Other.EnableShadowing ||
-        ValuePrinting         != Other.ValuePrinting ||
-        ResultEvaluation      != Other.ResultEvaluation ||
-        DynamicScoping        != Other.DynamicScoping ||
-        Debug                 != Other.Debug ||
-        CodeGeneration        != Other.CodeGeneration ||
-        CodeGenerationForModule != Other.CodeGenerationForModule ||
-        IgnorePromptDiags     != Other.IgnorePromptDiags ||
-        CheckPointerValidity  != Other.CheckPointerValidity ||
-        OptLevel              != Other.OptLevel ||
-        CodeCompletionOffset  != Other.CodeCompletionOffset;
+      return DeclarationExtraction != Other.DeclarationExtraction ||
+             EnableShadowing != Other.EnableShadowing ||
+             ValuePrinting != Other.ValuePrinting ||
+             ResultEvaluation != Other.ResultEvaluation ||
+             DynamicScoping != Other.DynamicScoping || Debug != Other.Debug ||
+             CodeGeneration != Other.CodeGeneration ||
+             CodeGenerationForModule != Other.CodeGenerationForModule ||
+             IgnorePromptDiags != Other.IgnorePromptDiags ||
+             CheckPointerValidity != Other.CheckPointerValidity ||
+             OptLevel != Other.OptLevel;
     }
   };
 } // end namespace cling

interpreter/cling/lib/Interpreter/IncrementalParser.cpp

@@ -860,9 +860,6 @@ namespace cling {
 
     Sema& S = getCI()->getSema();
 
-    const CompilationOptions& CO
-       = m_Consumer->getTransaction()->getCompilationOpts();
-
     // Recover resources if we crash before exiting this method.
     llvm::CrashRecoveryContextCleanupRegistrar<Sema> CleanupSema(&S);
 
@@ -902,12 +899,6 @@ namespace cling {
                                               0 /* mod time*/);
     SM.overrideFileContents(FE, std::move(MB));
     FID = SM.createFileID(FE, NewLoc, SrcMgr::C_User);
-    if (CO.CodeCompletionOffset != -1) {
-      // The completion point is set one a 1-based line/column numbering.
-      // It relies on the implementation to account for the wrapper extra line.
-      PP.SetCodeCompletionPoint(FE, 1/* start point 1-based line*/,
-                                CO.CodeCompletionOffset+1/* 1-based column*/);
-    }
 
     // NewLoc only used for diags.
     PP.EnterSourceFile(FID, /*DirLookup*/nullptr, NewLoc);
@@ -984,23 +975,6 @@ namespace cling {
                                                  std::error_code());
     }
 
-    if (CO.CodeCompletionOffset != -1) {
-#ifndef NDEBUG
-      Preprocessor& PP = m_CI->getPreprocessor();
-      SourceManager& SM = getCI()->getSourceManager();
-      assert((int)SM.getFileOffset(PP.getCodeCompletionLoc())
-             == CO.CodeCompletionOffset
-             && "Completion point wrongly set!");
-      assert(PP.isCodeCompletionReached()
-             && "Code completion set but not reached!");
-#endif
-
-      // Let's ignore this transaction:
-      m_Consumer->getTransaction()->setIssuedDiags(Transaction::kErrors);
-
-      return llvm::Error::success();
-    }
-
     // Process any TopLevelDecls generated by #pragma weak.
     for (Decl* D : S.WeakTopLevelDecls()) {
       DeclGroupRef DGR(D);

interpreter/cling/lib/Interpreter/Interpreter.cpp

@@ -984,8 +984,6 @@ namespace cling {
     size_t wrapPos = utils::getWrapPoint(wrapped, getCI()->getLangOpts());
     const std::string& Src = WrapInput(wrapped, wrapped, wrapPos);
 
-    CO.CodeCompletionOffset = offset + wrapPos;
-
     StateDebuggerRAII stateDebugger(this);
 
     // This triggers the FileEntry to be created and the completion