Skip to content

Conversation

@tomflinda
Copy link

Signed-off-by: chenwei.sun chenwei.sun@intel.com

…r the migrated codes in folder `guided_eigenValues_SYCLMigration/01_dpct_output`

Signed-off-by: chenwei.sun <chenwei.sun@intel.com>
@tomflinda tomflinda changed the title Fix linking error of multiple definition of 'computeGerschgorin' for the migrated codes in folder guided_eigenValues_SYCLMigration/01_dpct_output Fix linking error of multiple definition of 'computeGerschgorin' for sample eigenValues May 28, 2025
@tomflinda tomflinda changed the title Fix linking error of multiple definition of 'computeGerschgorin' for sample eigenValues Fix the linking error of multiple definition of 'computeGerschgorin' for sample eigenValues May 28, 2025
@jimmytwei
Copy link
Contributor

I'm not getting any linking errors compiling the code sample. I did update the header file to use "image.hpp" instead of <image.hpp> but didn't need to remove the other file.

Example output:

$ mkdir build; cd build; cmake ..; cmake -D INTEL_MAX_GPU=1 ..; make; make run
-- The C compiler identification is GNU 11.4.0
-- The CXX compiler identification is IntelLLVM 2025.2.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /opt/notebooks/user_home/intel/oneapi/compiler/2025.2/bin/icpx - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Default CMAKE_BUILD_TYPE not set using Release with Debug Info
-- Configuring done
-- Generating done
-- Build files have been written to: /opt/notebooks/user_home/oneAPI-samples/DirectProgramming/C++SYCL/SYCLMigration/guided_eigenValues_SYCLMigration/build
-- Configuring done
-- Generating done
-- Build files have been written to: /opt/notebooks/user_home/oneAPI-samples/DirectProgramming/C++SYCL/SYCLMigration/guided_eigenValues_SYCLMigration/build
[ 14%] Building CXX object 02_sycl_migrated/CMakeFiles/02_sycl_migrated.dir/Samples/2_Concepts_and_Techniques/eigenvalues/main.dp.cpp.o
[ 28%] Building CXX object 02_sycl_migrated/CMakeFiles/02_sycl_migrated.dir/Samples/2_Concepts_and_Techniques/eigenvalues/gerschgorin.cpp.o
[ 42%] Building CXX object 02_sycl_migrated/CMakeFiles/02_sycl_migrated.dir/Samples/2_Concepts_and_Techniques/eigenvalues/bisect_large.dp.cpp.o
[ 57%] Building CXX object 02_sycl_migrated/CMakeFiles/02_sycl_migrated.dir/Samples/2_Concepts_and_Techniques/eigenvalues/bisect_small.dp.cpp.o
[ 71%] Building CXX object 02_sycl_migrated/CMakeFiles/02_sycl_migrated.dir/Samples/2_Concepts_and_Techniques/eigenvalues/bisect_util.dp.cpp.o
[ 85%] Building CXX object 02_sycl_migrated/CMakeFiles/02_sycl_migrated.dir/Samples/2_Concepts_and_Techniques/eigenvalues/matlab.cpp.o
[100%] Linking CXX executable ../bin/02_sycl_migrated
Compilation from IR - skipping loading of FCL
Build succeeded.
Compilation from IR - skipping loading of FCL
Build succeeded.
Compilation from IR - skipping loading of FCL
Build succeeded.
[100%] Built target 02_sycl_migrated
Starting eigenvalues

Running on Intel(R) Data Center GPU Max 1100
Matrix size: 2048 x 2048
Precision: 0.000010
Iterations to be timed: 100
Result filename: 'eigenvalues.dat'
Gerschgorin interval: -2.894310 / 2.923303
Average time step 1: 5.769216 ms
Average time step 2, one intervals: 0.009240 ms
Average time step 2, mult intervals: 0.008500 ms
Average time TOTAL: 13.913960 ms
Test Succeeded!
Built target run

@jimmytwei jimmytwei closed this Jun 6, 2025
@tomflinda
Copy link
Author

@jimmytwei the default CMake build script is for folder SYCLMigration/guided_eigenValues_SYCLMigration/02_sycl_migrated , more detail, pls refer to

Current PR is targeted to fix the link issue in folder SYCLMigration/guided_eigenValues_SYCLMigration/01_dpct_output

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 this pull request may close these issues.

2 participants