Skip to content

Rename .cpp file to .cu for cpp_itfs#1826

Merged
valarLip merged 1 commit intoROCm:mainfrom
yuguo68:main
Jan 17, 2026
Merged

Rename .cpp file to .cu for cpp_itfs#1826
valarLip merged 1 commit intoROCm:mainfrom
yuguo68:main

Conversation

@yuguo68
Copy link
Contributor

@yuguo68 yuguo68 commented Jan 13, 2026

Motivation

Our build rules uses hipclang to compile .cu files, and normal clang for .cpp files. The .cpp files cpp_itfs all reference ck headers and cannot be compiled with normal clang.

Technical Details

Test Plan

Test Result

Submission Checklist

@yuguo68 yuguo68 requested a review from a team January 13, 2026 08:22
@yuguo68
Copy link
Contributor Author

yuguo68 commented Jan 13, 2026

@slippedJim to review, thanks!

@valarLip
Copy link
Collaborator

emmm could you please resolve the confict

@yuguo68
Copy link
Contributor Author

yuguo68 commented Jan 16, 2026

emmm could you please resolve the confict

done

@yuguo68
Copy link
Contributor Author

yuguo68 commented Jan 17, 2026

@valarLip could you help review? thanks!

@valarLip valarLip merged commit 63f588c into ROCm:main Jan 17, 2026
15 checks passed
yzhou103 pushed a commit that referenced this pull request Jan 28, 2026
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