[clang][cas] Make missing include-tree id an error

This was review feedback on a previous PR that somehow went missing.
Bring it back.

(cherry picked from commit f1ccf9d)
(cherry picked from commit 0b727cd)

Author: benlangmuir

clang/lib/Tooling/DependencyScanning/IncludeTreeActionController.cpp

@@ -392,11 +392,14 @@ Error IncludeTreeActionController::finalizeModuleBuild(
 
 Error IncludeTreeActionController::finalizeModuleInvocation(
     CompilerInvocation &CI, const ModuleDeps &MD) {
-  if (auto ID = MD.IncludeTreeID) {
-    configureInvocationForCaching(CI, CASOpts, std::move(*ID),
-                                  /*CASFSWorkingDir=*/"",
-                                  /*ProduceIncludeTree=*/true);
-  }
+  if (!MD.IncludeTreeID)
+    return llvm::createStringError(llvm::inconvertibleErrorCode(),
+                                   "missing include-tree for module '%s'",
+                                   MD.ID.ModuleName.c_str());
+
+  configureInvocationForCaching(CI, CASOpts, *MD.IncludeTreeID,
+                                /*CASFSWorkingDir=*/"",
+                                /*ProduceIncludeTree=*/true);
 
   DepscanPrefixMapping::remapInvocationPaths(CI, PrefixMapper);
   return Error::success();