Skip to content

[SYCL][Driver] Windows output with -fsycl-link not generating expecte… #999

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

Merged

Conversation

mdtoguchi
Copy link
Contributor

…d output

When specifying output options with -fsycl-link -fintelfpga on Windows, the
named output is not used. This is due to the output type for -fsycl-link
being an archive type not being recognized when generating output file names.

Also address a problem seen where use of /Fo was using that name for the
intermediate unbundling step.

Signed-off-by: Michael D Toguchi [email protected]

Copy link
Contributor

@AGindinson AGindinson left a comment

Choose a reason for hiding this comment

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

Besides the code comments, I'd suggest a slight rewording for the commit message, e.g.:
"Fix output options behavior for -fsycl-link on Windows"

When specifying output options with -fsycl-link -fintelfpga on Windows, the
named output is not used.  This is due to the output type for -fsycl-link
being an archive type not being recognized when generating output file names.

Also address a problem seen where use of /Fo was using that name for the
intermediate unbundling step.

Signed-off-by: Michael D Toguchi <[email protected]>
@mdtoguchi mdtoguchi force-pushed the private/mdtoguchi/fsycl-link-win-output branch from 83e1506 to b3484c7 Compare January 13, 2020 18:55
Copy link
Contributor

@AGindinson AGindinson left a comment

Choose a reason for hiding this comment

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

Would the suggested changes be too much? It just seems to be guarding against more cases alike in advance.

When the output file is specified by the user, that output file should not
be used for the intermediate unbundling step.

Signed-off-by: Michael D Toguchi <[email protected]>
Copy link
Contributor

@AGindinson AGindinson left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM!

@romanovvlad romanovvlad merged commit 67b24d4 into intel:sycl Jan 16, 2020
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