Skip to content

[SYCL][FPGA] Prepare future implementation of experimental pipe properties #5886

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

pcolberg
Copy link
Contributor

@pcolberg pcolberg commented Mar 25, 2022

Add a template parameter to pass compile-time constant properties to
sycl::ext::intel::experimental::pipe, and redeclare pipe implementation
as a specialization when the properties list is empty; otherwise, when
the properties list is non-empty, throw a descriptive compiler error.

The unnamed template parameter allows implementation of the case where
the properties list is non-empty in a downstream, FPGA-specific header,
before finalizing pipe properties for inclusion into this header.

See draft specification in #5839

@GarveyJoe @rho180 @sherry-yuan @bsyrowik @tiwaria1 @ajaykumarkannan

@pcolberg pcolberg self-assigned this Mar 25, 2022
@pcolberg pcolberg requested a review from a team as a code owner March 25, 2022 00:14
@pcolberg pcolberg force-pushed the sycl-ext-intel-experimental-pipes-properties branch 2 times, most recently from eb65fa6 to b640130 Compare March 25, 2022 01:06
…rties

Add a template parameter to pass compile-time constant properties to
sycl::ext::intel::experimental::pipe, and redeclare pipe implementation
as a specialization when the properties list is empty; otherwise, when
the properties list is non-empty, throw a descriptive compiler error.

The unnamed template parameter allows implementation of the case where
the properties list is non-empty in a downstream, FPGA-specific header,
before finalizing pipe properties for inclusion into this header.

See draft specification in #5838
@pcolberg pcolberg force-pushed the sycl-ext-intel-experimental-pipes-properties branch from b640130 to c521a84 Compare March 25, 2022 01:27
@pcolberg
Copy link
Contributor Author

Same unrelated test failures as outlined in #5766 (comment).

Copy link
Contributor

@gmlueck gmlueck left a comment

Choose a reason for hiding this comment

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

This does not match the current wording in the draft specification #5839, but it is in line with our offline discussions. I have no concerns with this PR, but someone from the implementation team should review.

@pcolberg pcolberg removed the request for review from cperkinsintel March 25, 2022 17:47
@pcolberg
Copy link
Contributor Author

Hi @intel/llvm-reviewers-runtime, since @cperkinsintel is unavailable for review, it would be great if you could take over the review in time for the next release.

@pcolberg
Copy link
Contributor Author

@intel/llvm-reviewers-runtime could you take a quick look to ensure this gets in before code complete?

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

@pcolberg
Copy link
Contributor Author

Thanks @s-kanaev and @steffenlarsen for the quick review 🙂

@intel/llvm-gatekeepers could you merge in time for code complete?

@bader bader changed the title [SYCL][FPGA] prepare future implementation of experimental pipe properties [SYCL][FPGA] Prepare future implementation of experimental pipe properties Mar 30, 2022
@bader
Copy link
Contributor

bader commented Mar 30, 2022

@intel/dpcpp-esimd-reviewers, FYI. 4 failing tests are failing on ESIMD emulator - https://github.com/intel/llvm/runs/5685843530?check_suite_focus=true. I don't think these failures are related to this PR, so I'm going to merge it.

@bader bader merged commit 48c8cc1 into intel:sycl Mar 30, 2022
@pcolberg pcolberg deleted the sycl-ext-intel-experimental-pipes-properties branch March 30, 2022 15:33
alexbatashev added a commit to alexbatashev/llvm that referenced this pull request Apr 2, 2022
* sycl: (3343 commits)
  [SYCL][L0] Disable round-robin submissions to multiple CCSs (intel#5945)
  [SYCL][CUDA] Don't link pi_cuda against libsycl (intel#5908)
  [CI] Disable -Werror by default (intel#5889)
  [BuildBot] Uplift CPU/FPGAEMU RT version to 2022.13.3.0.16 (intel#5883)
  [SYCL][CUDA][libclc] Add support for atomic fp exchange and compare exchange (intel#5937)
  [SYCL] Fix device code outlining for static local variables (intel#5915)
  [SYCL][NFC] Refactor plugin CMakeLists.txt (intel#5799)
  [SPIR-V][Doc] Add JointMatrixWorkItemLengthINTEL instruction to joint matrix extension (intel#5781)
  [SYCL] Expand device_global map and make initialization order agnostic (intel#5902)
  [SYCL][CUDA] Add IPSCCP pass to O0 by default (intel#5900)
  [ESIMD] Disable ABI changes warnings in host compiler. (intel#5931)
  [SYCL] Make properties constructor constexpr (intel#5928)
  [NFC][SYCL] Fix static analysis warning (intel#5933)
  [CODEOWNERS][NFC] Assign code owners for CI scripts (intel#5873)
  [SYCL] Store the kernel object size in the integration header (intel#5862)
  [SYCL][ESIMD] Change esimd-verifier logic for detecting valid SYCL calls (intel#5914)
  [SYCL][CUDA][DOC] GettingStartedGuide.md to recommend cuda 11.6 (intel#5917)
  [SYCL][L0] Move command list cache usage under mutex (intel#5874)
  [SYCL][FPGA] Prepare future implementation of experimental pipe properties (intel#5886)
  [CI] Roll back intel driver to the latest version (intel#5925)
  ...
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.

5 participants