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

Rename common_clang to opencl_clang #404

Merged
merged 1 commit into from
Mar 3, 2023

Conversation

wenju-he
Copy link
Contributor

@wenju-he wenju-he commented Mar 1, 2023

Rename COMMON_CLANG as well.

common_clang is old name, so it is better to rename it.

@wenju-he wenju-he requested review from haonanya, cdai2 and fzou1 March 1, 2023 02:30
@wenju-he
Copy link
Contributor Author

wenju-he commented Mar 3, 2023

ping

@haonanya
Copy link

haonanya commented Mar 3, 2023

lgtm, @hewj03 , thank you!
@cdai2 , @fzou1 please review it as well.

Copy link
Contributor

@fzou1 fzou1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@@ -101,10 +101,10 @@ else (WIN32)
endif (WIN32)

# set that name of the main output file as a target name
if (NOT DEFINED COMMON_CLANG_LIBRARY_NAME)
set(COMMON_CLANG_LIBRARY_NAME "opencl-clang")
if (NOT DEFINED OPENCL_CLANG_LIBRARY_NAME)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As we rename this macro. Do we need to notify IGC compiler team? Maybe they use "COMMON_CLANG_LIBRARY_NAME" during their compilation.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

library name in igc build is libopencl-clang.so

@wenju-he wenju-he merged commit 98d206e into intel:main Mar 3, 2023
@wenju-he wenju-he deleted the rename-COMMON_CLANG branch March 3, 2023 05:54
@intel intel deleted a comment from wenju-he May 11, 2023
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.

4 participants