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

[Driver][SYCL] Remove COFF object format designator for Windows devic… #1574

Merged
merged 1 commit into from
Apr 23, 2020

Conversation

mdtoguchi
Copy link
Contributor

…e compiles

When compiling with clang with the Windows target or with clang-cl, the
device triple used was different. There seems to be no reason for this
difference and only causes mismatches and confusion when using the
different drivers even though they are both targeting Windows. Effectively
remove this format designator when working with the device triple.

Signed-off-by: Michael D Toguchi michael.d.toguchi@intel.com

@mdtoguchi mdtoguchi requested a review from AGindinson as a code owner April 22, 2020 23:49
…e compiles

When compiling with clang with the Windows target or with clang-cl, the
device triple used was different.  There seems to be no reason for this
difference and only causes mismatches and confusion when using the
different drivers even though they are both targeting Windows.  Effectively
remove this format designator when working with the device triple.

Signed-off-by: Michael D Toguchi <michael.d.toguchi@intel.com>
@mdtoguchi mdtoguchi force-pushed the private/mdtoguchi/windows-coff-device branch from a0d7e4b to 9669d23 Compare April 22, 2020 23:53
@mdtoguchi mdtoguchi requested a review from sndmitriev April 22, 2020 23:54
Copy link
Contributor

@sndmitriev sndmitriev left a comment

Choose a reason for hiding this comment

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

Looks good.

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.

LGTM, thanks!

@bader bader merged commit 0052d08 into intel:sycl Apr 23, 2020
alexbatashev pushed a commit to alexbatashev/llvm that referenced this pull request Apr 29, 2020
…versioning

* origin/sycl:
  [XPTI][Framework] Reference implementation of the Xpti framework to be used with instrumentation in SYCL (intel#1557)
  [SYCL] Initial ABI checks implementation (intel#1528)
  [SYCL] Support connection with multiple plugins (intel#1490)
  [SYCL] Add a new header file with the reduction class definition (intel#1558)
  [SYCL] Add test for SYCL kernels with accessor and spec constant (intel#1536)
  [SYCL][CUDA] Move interop tests (intel#1570)
  [Driver][SYCL] Remove COFF object format designator for Windows device compiles (intel#1574)
  [SYCL] Fix conflicting visibility attributes (intel#1571)
  [SYCL][DOC] Update the SYCL Runtime Interface document with design details (intel#680)
  [SYCL] Improve image accessors support on a host device (intel#1502)
  [SYCL] Make queue's non-USM event ownership temporary (intel#1561)
  [SYCL] Added support of rounding modes for non-host devices (intel#1463)
  [SYCL] SemaSYCL significant refactoring (intel#1517)
  [SYCL] Support 0-dim accessor in handler::copy(accessor, accessor) (intel#1551)
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.

4 participants