[PATCH] D127318: [cmake] Add missing dependencies to objlib in add_llvm_executable.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: llvm/cmake/modules/AddLLVM.cmake:966
+    # Add dependencies also to obj_name
+    # CMake issue 14747 --  add_dependencies() might be ignored to obj_name's user.
+    foreach(objlib ${obj_name})
----------------
beanz wrote:
> Unless I'm misunderstanding the behavior issues you're seeing here are a bug in our code not CMake.  IIUC, the issue you're seeing is that since the objlib doesn't depend on the tablegen targets, it is likely failing to build or building out of order. That's clearly a bug in our code, not CMake.
> 
> I know that you copied that comment, but it doesn't seem relevant here, and I think it is describing a situation that isn't actually a bug in CMake.
Thanks, I agree that the comment here is a bit out of place. I've removed it in the committed version.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127318/new/

https://reviews.llvm.org/D127318