Fix global handling in type state analysis (#651)

* fix global handling in type state analysis

* Fix error due to update from dev

* fix killing of globals: only kill if no declaration-only func is called

---------

Co-authored-by: Martin Mory <linuxfan91@googlemail.com>
Co-authored-by: Fabian Schiebel <fabian.schiebel@iem.fraunhofer.de>

Author: 3 people

lib/PhasarLLVM/DataFlow/IfdsIde/Problems/IDETypeStateAnalysis.cpp

@@ -182,13 +182,16 @@ auto IDETypeStateAnalysisBase::getCallToRetFlowFunction(
     n_t CallSite, n_t /*RetSite*/, llvm::ArrayRef<f_t> Callees)
     -> FlowFunctionPtrType {
   const auto *CS = llvm::cast<llvm::CallBase>(CallSite);
+  bool DeclarationOnlyCalleeFound = false;
   for (const auto *Callee : Callees) {
     std::string DemangledFname = llvm::demangle(Callee->getName().str());
     // Generate the return value of factory functions from zero value
     if (isFactoryFunction(DemangledFname)) {
       return this->generateFromZero(CS);
     }
 
+    DeclarationOnlyCalleeFound |= Callee->isDeclaration();
+
     /// XXX: Revisit this:
 
     // Handle all functions that are not modeld with special semantics.
@@ -209,6 +212,10 @@ auto IDETypeStateAnalysisBase::getCallToRetFlowFunction(
       }
     }
   }
+  if (!DeclarationOnlyCalleeFound) {
+    return killFlowIf(
+        [](d_t Source) { return llvm::isa<llvm::Constant>(Source); });
+  }
   return identityFlow();
 }