Skip to content

Conversation

@Rohan138
Copy link
Contributor

Motivation

https://github.com/ROCm/aiter/pull/830/files/b79c6d3eeaf4588ad495a5c48b0d35ef9ec3ed58 explicitly copies the headers from the ck gemm include folders e.g. aiter/csrc/ck_batched_gemm_a8w8/include into {prebuild_dir}/include.

However, #1159 added the corresponding include directories for each module into the extra_include field in optCompilerConfig.json, so we don't actually need the redundant copy; the folders are already included in the final build as part of prebuild_link_param["extras_include"] here.

Mildly relevant copilot comment: #1423 (comment)

Technical Details

Test Plan

Test Result

Submission Checklist

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