Skip to content

Conversation

@jinge90
Copy link
Contributor

@jinge90 jinge90 commented Feb 25, 2022

On Windows, users will encounter host link error saying
"ITERATOR DEBUG LEVEL MISMATCH" when building programs
with /MDd when libsycl-fallback-cassert.obj is involved.
In fact, the host obj extracted from libdevice is an
empty file and we can use "-D_ALLOW_ITERATOR_DEBUG_LEVEL"
to bypass this issue.

Signed-off-by: jinge90 ge.jin@intel.com

@jinge90 jinge90 requested a review from a team as a code owner February 25, 2022 03:40
@jinge90 jinge90 requested a review from s-kanaev February 25, 2022 03:40
On Windows, users will encounter host link error saying
"ITERATOR DEBUG LEVEL MISMATCH" when building programs
with /MDd when libsycl-fallback-cassert.obj is involved.
In fact, the host obj extracted from libdevice is an
empty file and we can use "-D_ALLOW_ITERATOR_DEBUG_LEVEL"
to bypass this issue.

Signed-off-by: jinge90 <ge.jin@intel.com>
@jinge90
Copy link
Contributor Author

jinge90 commented Mar 1, 2022

Hi, @s-kanaev
Could you help review this PR?
Thanks very much.

Copy link
Contributor

@s-kanaev s-kanaev left a comment

Choose a reason for hiding this comment

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

LGTM

Though, is it possible to have a test for such a behaviour?

@jinge90 jinge90 requested a review from bader March 4, 2022 08:46
@jinge90
Copy link
Contributor Author

jinge90 commented Mar 4, 2022

Hi, @bader and @s-kanaev
Could check if this patch can be merged?

Thanks very much.

@bader
Copy link
Contributor

bader commented Mar 5, 2022

Though, is it possible to have a test for such a behaviour?

@jinge90, there is a question from @s-kanaev about testing this change. Could you answer this, please?

@bader bader merged commit 93b573a into intel:sycl Mar 7, 2022
@s-kanaev
Copy link
Contributor

s-kanaev commented Mar 9, 2022

The testing question was resolved in e-mail thread.
@bader, I'll cc the thread to you.

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