Skip to content

Conversation

aarongreig
Copy link
Contributor

Reverts #18764

Looks like the naive level zero implementation now results in a performance regression, I think it's best to revert until l0 can better match the new interface.

@aarongreig aarongreig requested review from a team as code owners July 31, 2025 15:35
@aarongreig
Copy link
Contributor Author

@intel/llvm-gatekeepers please merge

@sarnex sarnex merged commit d25d6d6 into sycl Aug 4, 2025
48 of 49 checks passed
@bader bader deleted the revert-18764-aaron/enqueueKernelWithArgs branch August 5, 2025 17:47
@againull
Copy link
Contributor

againull commented Sep 22, 2025

@aarongreig @pbalcer Do you have the info/link to the issue regarding the L0 perf regression? I was wondering when this PR will be relanded.

@pbalcer
Copy link
Contributor

pbalcer commented Sep 23, 2025

@aarongreig @pbalcer Do you have the info/link to the issue regarding the L0 perf regression? I was wondering when this PR will be relanded.

The person that took this over from @aarongreig was on vacation, but we will be picking this up soon.

staniewzki added a commit to staniewzki/llvm that referenced this pull request Sep 23, 2025
staniewzki added a commit to staniewzki/llvm that referenced this pull request Sep 25, 2025
ldorau added a commit to ldorau/llvm that referenced this pull request Sep 25, 2025
ldorau added a commit to ldorau/llvm that referenced this pull request Oct 8, 2025
ldorau added a commit to ldorau/llvm that referenced this pull request Oct 8, 2025
ldorau added a commit to ldorau/llvm that referenced this pull request Oct 8, 2025
…ch in one call." (intel#19661)"

This reverts commit d25d6d6.

Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
ldorau added a commit to ldorau/llvm that referenced this pull request Oct 8, 2025
…ch in one call." (intel#19661)"

This reverts commit d25d6d6.

Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
ldorau added a commit to ldorau/llvm that referenced this pull request Oct 8, 2025
…ch in one call." (intel#19661)"

This reverts commit d25d6d6.

Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
ldorau added a commit to ldorau/llvm that referenced this pull request Oct 9, 2025
…ch in one call." (intel#19661)"

This reverts commit d25d6d6.

Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
ldorau added a commit to ldorau/llvm that referenced this pull request Oct 9, 2025
…ch in one call." (intel#19661)"

This reverts commit d25d6d6.

Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
ldorau added a commit to ldorau/llvm that referenced this pull request Oct 10, 2025
…ch in one call." (intel#19661)"

This reverts commit d25d6d6.

Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
ldorau added a commit to ldorau/llvm that referenced this pull request Oct 10, 2025
…ch in one call." (intel#19661)"

This reverts commit d25d6d6.

Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
ldorau added a commit to ldorau/llvm that referenced this pull request Oct 14, 2025
…ch in one call." (intel#19661)"

This reverts commit d25d6d6.

Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
ldorau added a commit to ldorau/llvm that referenced this pull request Oct 15, 2025
…ch in one call." (intel#19661)"

This reverts commit d25d6d6.

Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
ldorau added a commit to ldorau/llvm that referenced this pull request Oct 16, 2025
…ch in one call." (intel#19661)"

This reverts commit d25d6d6.

Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
ldorau added a commit to ldorau/llvm that referenced this pull request Oct 16, 2025
…ch in one call." (intel#19661)"

This reverts commit d25d6d6.

Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
ldorau added a commit to ldorau/llvm that referenced this pull request Oct 17, 2025
…ch in one call." (intel#19661)"

This reverts commit d25d6d6.

Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
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.

8 participants