default `install` target has missing dependencies

[ldrumm]

Running ninja install does not correctly complete the install. The install target should correctly track all dependencies without requiring another target to be run.

$ python ../buildbot/configure.py --cuda -o . && ninja install
[ ...]
CMake Error at tools/libdevice/cmake_install.cmake:46 (file):
  file INSTALL cannot find
  "/home/luke/src/upstream/llvm-project/build/lib/libsycl-itt-stubs.o": No
  such file or directory.
Call Stack (most recent call first):
  tools/cmake_install.cmake:147 (include)
  cmake_install.cmake:80 (include)

ninja deploy-sycl-toolchain && ninja install creates the missing dependency and allows the install to complete. This indicates that the install target for ALL is missing an explicit dependency

Local config:
OS: Debian sid gcc=12, cmake=3.24.2, ninja=1.11.1, cuda=11.7.1-1

$ git rev-parse --short HEAD
70b67671a342

[matsbror]

Thanks for posting the workaround, it helped me get past it.

[keryell]

Just hitting this today on the sycl-mlir branch. Probably still the same on sycl branch.
As suggested, the following work-around worked:

# Try install from scratch after configuration
cmake --build $DPCPP_HOME/llvm/build --target install
CMake Error at tools/libdevice/cmake_install.cmake:46 (file):
  file INSTALL cannot find
  "/storage/rkeryell/SYCL/sycl-mlir/llvm/build/./lib/libsycl-itt-stubs.o": No  such file or directory.
Call Stack (most recent call first):
  tools/cmake_install.cmake:562 (include)
  cmake_install.cmake:122 (include)
# Work-around
cmake --build $DPCPP_HOME/llvm/build --target=deploy-sycl-toolchain
cmake --build $DPCPP_HOME/llvm/build --target install

[bader]

We added deploy-sycl-toolchain target to install DPC++.

[keryell]

We added deploy-sycl-toolchain target to install DPC++.

Cool. But I am confused. Do you mean into the sycl branch recently and this should be closed?
Or that it is an old story on the sycl branch, this issue was forgotten and for some reason it did not percolate into the sycl-mlir branch?

[bader]

We use deploy-sycl-toolchain target for years, I don't think we ever used install target to deploy DPC++ compiler.
I'm not aware of sycl-mlir branch changes regarding the deployment procedure.

[KornevNikita]

Hi! There have been no updates for at least the last 60 days, though the ticket has assignee(s).

@ldrumm, could I ask you to take one of the following actions? :)

• Please provide an update if you have any (or just a small comment if you don't have any yet).
• OR mark this issue with the 'confirmed' label if you have confirmed the problem/request and our team should work on it.
• OR close the issue if it has been resolved.
• OR take any other suitable action.

Thanks!

[ldrumm]

Why do we need to do those things? If nothing has changed, I don't see why the ticket needs changing

[KornevNikita]

Hello @ldrumm,

Why do we need to do those things? If nothing has changed, I don't see why the ticket needs changing

Because the issue has assignee.

I assume that the assignee is working on this or planning to. Since there are no updates I just wanted to check if you're looking into this. If no - you could unassign it and mark with the 'confirmed' label, so we would take a look by ourselves.

[ldrumm]

On Tue May 21, 2024 at 9:49 AM BST, Nikita Kornev wrote: Hello @ldrumm, > Why do we need to do those things? If nothing has changed, I don't see why the ticket needs changing Because the issue has assignee.

I'm the assignee

[github-actions]

Hi! There have been no updates for at least the last 60 days, though the issue has assignee(s).

@ldrumm, could you please take one of the following actions:

• provide an update if you have any
• unassign yourself if you're not looking / going to look into this issue
• mark this issue with the 'confirmed' label if you have confirmed the problem/request and our team should work on it
• close the issue if it has been resolved
• take any other suitable action.

Thanks!

[matthew-olson-intel]

I'm getting this error, too, using the latest master and building with:

python ./buildbot/configure.py \
  -t Debug \
  --cmake-opt=-DCMAKE_CXX_FLAGS="-fno-omit-frame-pointer -mno-omit-leaf-frame-pointer" \
  --cmake-opt=-DCMAKE_C_FLAGS="-fno-omit-frame-pointer -mno-omit-leaf-frame-pointer" \
  --cmake-opt=-DCMAKE_INSTALL_PREFIX="${PREFIX}" \
  --cmake-opt=-DSYCL_ENABLE_PLUGINS="level_zero" \
  --cmake-opt=-DLLVM_ENABLE_PROJECTS="opencl;openmp;clang"
cd build
ninja deploy-sycl-toolchain

[bader]

@matthew-olson-intel, please, try using sycl branch instead of master.