[PATCH] D153166: AMDGPU: Remove add_dependencies calls from CMakeLists.txt

Tom Stellard via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 16 13:05:40 PDT 2023


tstellar created this revision.
tstellar added a reviewer: arsenm.
Herald added subscribers: foad, kerbowa, hiraditya, tpr, dstuttard, yaxunl, jvesely, kzhuravl.
Herald added a project: All.
tstellar requested review of this revision.
Herald added a subscriber: wdng.
Herald added a project: LLVM.

These are redundant.  The same dependencies are being added as part
of the add_llvm_component_library() call.  I confirmed this by diff'ing
the build.ninja files before and after the change and saw no change.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D153166

Files:
  llvm/lib/Target/AMDGPU/AsmParser/CMakeLists.txt
  llvm/lib/Target/AMDGPU/Disassembler/CMakeLists.txt
  llvm/lib/Target/AMDGPU/MCA/CMakeLists.txt
  llvm/lib/Target/AMDGPU/MCTargetDesc/CMakeLists.txt


Index: llvm/lib/Target/AMDGPU/MCTargetDesc/CMakeLists.txt
===================================================================
--- llvm/lib/Target/AMDGPU/MCTargetDesc/CMakeLists.txt
+++ llvm/lib/Target/AMDGPU/MCTargetDesc/CMakeLists.txt
@@ -25,5 +25,3 @@
   ADD_TO_COMPONENT
   AMDGPU
   )
-
-add_dependencies(LLVMAMDGPUDesc LLVMAMDGPUUtils)
Index: llvm/lib/Target/AMDGPU/MCA/CMakeLists.txt
===================================================================
--- llvm/lib/Target/AMDGPU/MCA/CMakeLists.txt
+++ llvm/lib/Target/AMDGPU/MCA/CMakeLists.txt
@@ -15,5 +15,3 @@
   ADD_TO_COMPONENT
   AMDGPU
   )
-
-add_dependencies(LLVMAMDGPUTargetMCA LLVMAMDGPUUtils)
Index: llvm/lib/Target/AMDGPU/Disassembler/CMakeLists.txt
===================================================================
--- llvm/lib/Target/AMDGPU/Disassembler/CMakeLists.txt
+++ llvm/lib/Target/AMDGPU/Disassembler/CMakeLists.txt
@@ -15,5 +15,3 @@
   ADD_TO_COMPONENT
   AMDGPU
   )
-
-add_dependencies(LLVMAMDGPUDisassembler AMDGPUCommonTableGen LLVMAMDGPUUtils)
Index: llvm/lib/Target/AMDGPU/AsmParser/CMakeLists.txt
===================================================================
--- llvm/lib/Target/AMDGPU/AsmParser/CMakeLists.txt
+++ llvm/lib/Target/AMDGPU/AsmParser/CMakeLists.txt
@@ -14,5 +14,3 @@
   ADD_TO_COMPONENT
   AMDGPU
   )
-
-add_dependencies(LLVMAMDGPUAsmParser LLVMAMDGPUUtils)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D153166.532275.patch
Type: text/x-patch
Size: 1367 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230616/3bd67027/attachment.bin>


More information about the llvm-commits mailing list