Skip to content

add -fclang-abi-compat=17 to HIP_HIPCC_FLAGS#1377

Merged
pruthvistony merged 1 commit intorocm6.2_internal_testingfrom
rocm6.2_internal_testing_fix_clang_abi_compat
Mar 22, 2024
Merged

add -fclang-abi-compat=17 to HIP_HIPCC_FLAGS#1377
pruthvistony merged 1 commit intorocm6.2_internal_testingfrom
rocm6.2_internal_testing_fix_clang_abi_compat

Conversation

@jeffdaily
Copy link
Collaborator

C++20 mangling rules were recently added to hip-clang. This flag maintains compatibility since pytorch is at C++17. Otherwise the linker fails.

C++20 mangling rules were recently added to hip-clang. This flag
maintains compatibility since pytorch is at C++17. Otherwise the linker
fails.
@jeffdaily
Copy link
Collaborator Author

Copy link

@shiltian shiltian left a comment

Choose a reason for hiding this comment

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

LG

@pruthvistony pruthvistony merged commit de739ad into rocm6.2_internal_testing Mar 22, 2024
pruthvistony pushed a commit that referenced this pull request Apr 23, 2024
C++20 mangling rules were recently added to hip-clang. This flag
maintains compatibility since pytorch is at C++17. Otherwise the linker
fails.
jeffdaily added a commit that referenced this pull request Apr 24, 2024
C++20 mangling rules were recently added to hip-clang. This flag
maintains compatibility since pytorch is at C++17. Otherwise the linker
fails.
pruthvistony pushed a commit that referenced this pull request Apr 29, 2024
C++20 mangling rules were recently added to hip-clang. This flag
maintains compatibility since pytorch is at C++17. Otherwise the linker
fails.
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.

3 participants