Skip to content

[Driver][SYCL] Keep the option value of -o option as the output direc… - #3912

Merged
bader merged 1 commit into
intel:syclfrom
qichaogu:output-report-dir
Jun 12, 2021
Merged

[Driver][SYCL] Keep the option value of -o option as the output direc…#3912
bader merged 1 commit into
intel:syclfrom
qichaogu:output-report-dir

Conversation

@qichaogu

@qichaogu qichaogu commented Jun 10, 2021

Copy link
Copy Markdown
Contributor

…tory name

When enabling FPGA AOT, the name of the output project report is not correct when using -o.
Only when user does not use "-o" option, the output directory name is from the
source file and the original extension is replaced with ".prj".
However, if user uses "-o" option, the output directory name should be the value
of the option with extention ".prj".

Signed-off-by: Qichao Gu qichao.gu@intel.com

…tory name

Signed-off-by: Qichao Gu <qichao.gu@intel.com>
@mdtoguchi

Copy link
Copy Markdown
Contributor

Could you add additional details to the body of your commit message to what you are doing? Some additional detail of what you are fixing to clarify the summary.

@qichaogu

Copy link
Copy Markdown
Contributor Author

Additional details updated.

@mdtoguchi mdtoguchi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@bader
bader merged commit 18ac172 into intel:sycl Jun 12, 2021
@qichaogu
qichaogu deleted the output-report-dir branch August 4, 2021 02:35
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