Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

libsycl.so-gdb.py installed with wrong name in wrong directory #1636

Closed
cgleggett opened this issue May 3, 2020 · 1 comment · Fixed by #1645
Closed

libsycl.so-gdb.py installed with wrong name in wrong directory #1636

cgleggett opened this issue May 3, 2020 · 1 comment · Fixed by #1645
Assignees

Comments

@cgleggett
Copy link

with 95ded99 libsycl.so-gdb.py is getting installed as
INST_DIR//SRC_DIR/sycl/xmethods/$<TARGET_FILE_NAME:sycl>-gdb.py

where INST_DIR is the value of CMAKE_INSTALL_PREFIX, SRC_DIR is the location of the llvm source, and $<TARGET_FILE_NAME:sycl> is actually that exact string.

this got messed up somewhere between bb73d92 and 95ded99

config.py options were --cuda --shared-libs -o . -t release --no-werror

@bader
Copy link
Contributor

bader commented May 3, 2020

Installation script is located here: https://github.com/intel/llvm/blob/sycl/sycl/CMakeLists.txt#L211 and was recently modified by @alexbatashev.
@alexbatashev, could you take a look, please?

Additional note: I would refactor the cmake files and move the parts related to libsyc.so-gdb.py to https://github.com/intel/llvm/tree/sycl/sycl/xmethods.

Tagging @mihails-strasuns-intel to keep him in the loop.

alexbatashev pushed a commit to alexbatashev/llvm that referenced this issue May 6, 2020
Fixes intel#1636

Signed-off-by: Alexander Batashev <alexander.batashev@intel.com>
bader pushed a commit that referenced this issue May 7, 2020
Fixes #1636

Signed-off-by: Alexander Batashev <alexander.batashev@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants