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

[SYCLomatic] Fix incorrect filter for option "-optf" and duplicate compile commands in compilation database due to working directory ignoring for intercept-build tool #1311

Merged
merged 3 commits into from
Oct 9, 2023

Conversation

tomflinda
Copy link
Contributor

  1. Fix incorrect process for option "-optf" short for “--options-file”
  2. Fix incorrect working directory for compile command like “cd inner_dir && nvcc -o foo foo.cu”

…pt-build toola

1. Fix incorrect process for option -optf short for “--options-file”
2. Fix incorrect working directory for compile command like “cd inner_dir && nvcc -o foo foo.cu”

Signed-off-by: chenwei.sun <[email protected]>
@tomflinda tomflinda requested a review from a team as a code owner September 20, 2023 00:40
@tomflinda tomflinda changed the title [SYCLomatic] Fix incorrect compilation database generation for intercept-build tool [SYCLomatic][Draft] Fix incorrect compilation database generation for intercept-build tool Sep 20, 2023
@tomflinda
Copy link
Contributor Author

behavior test for this PR has been submitted to SYCLomatic-test repo: oneapi-src/SYCLomatic-test#474

@tomflinda tomflinda changed the title [SYCLomatic][Draft] Fix incorrect compilation database generation for intercept-build tool [SYCLomatic]Fix incorrect compilation database generation for intercept-build tool Sep 21, 2023
@tomflinda tomflinda changed the title [SYCLomatic]Fix incorrect compilation database generation for intercept-build tool [SYCLomatic] Fix incorrect filter for option "-optf" and duplicate compile commands due to working directory skipping in compilation database generation for intercept-build tool Oct 9, 2023
@tomflinda tomflinda changed the title [SYCLomatic] Fix incorrect filter for option "-optf" and duplicate compile commands due to working directory skipping in compilation database generation for intercept-build tool [SYCLomatic] Fix incorrect filter for option "-optf" and duplicate compile commands in compilation database due to working directory ignoring for intercept-build tool Oct 9, 2023
Copy link
Contributor

@zhimingwang36 zhimingwang36 left a comment

Choose a reason for hiding this comment

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

LGTM

@zhimingwang36 zhimingwang36 merged commit 1b13358 into oneapi-src:SYCLomatic Oct 9, 2023
ShengchenJ pushed a commit to ShengchenJ/SYCLomatic that referenced this pull request Sep 27, 2024
…on database generation when source file builds with "-optf" of intercept-build tool (oneapi-src#474)

Signed-off-by: chenwei.sun <[email protected]>
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