================
@@ -437,27 +437,21 @@ function(add_libclc_builtin_set)
# Non-SPIR-V targets add an extra step to optimize the bytecode
set( builtins_opt_lib_tgt builtins.opt.${ARG_ARCH_SUFFIX} )
- add_custom_command( OUTPUT
${LIBCLC_ARCH_OBJFILE_DIR}/${builtins_opt_lib_tgt}.bc
- COMMAND ${opt_exe} ${ARG_OPT_FLAGS} -o
${LIBCLC_ARCH_OBJFILE_DIR}/${builtins_opt_lib_tgt}.bc
+ set( obj_suffix ${ARG_ARCH_SUFFIX}.bc )
+ set( libclc_builtins_lib ${LIBCLC_OUTPUT_LIBRARY_DIR}/${obj_suffix} )
+
+ add_custom_command( OUTPUT ${libclc_builtins_lib}
+ COMMAND ${opt_exe} ${ARG_OPT_FLAGS} -o ${libclc_builtins_lib}
${builtins_link_lib}
DEPENDS ${opt_target} ${builtins_link_lib} ${builtins_link_lib_tgt}
)
add_custom_target( ${builtins_opt_lib_tgt}
- ALL DEPENDS ${LIBCLC_ARCH_OBJFILE_DIR}/${builtins_opt_lib_tgt}.bc
+ ALL DEPENDS ${libclc_builtins_lib}
----------------
wenju-he wrote:
Below `prepare-${obj_suffix}` target also depends on ${libclc_builtins_lib}.
Having two targets depend on the same file may cause race condition.
Also, rename `prepare` target since `prepare` step is removed?
https://github.com/llvm/llvm-project/pull/174447
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
