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

LIT XFAIL spec_const tests passing with Intel OpenCL GPU 20.19.16754 #1873

Closed
bjoernknafla opened this issue Jun 11, 2020 · 4 comments · Fixed by #1875
Closed

LIT XFAIL spec_const tests passing with Intel OpenCL GPU 20.19.16754 #1873

bjoernknafla opened this issue Jun 11, 2020 · 4 comments · Fixed by #1875

Comments

@bjoernknafla
Copy link
Contributor

bjoernknafla commented Jun 11, 2020

After noticing in buildbot/dependency.conf to update to the Intel OpenCL GPU driver 20.19.16754 I am now seeing two LIT XFAILs for OpenCL pass:

cd <BUILD-DIR>
./bin/llvm-lit -a -v tools/sycl/test --param SYCL_BE=PI_OPENCL --filter=.*spec.*

...

Unexpectedly Passed Tests (2):
  SYCL :: spec_const/spec_const_hw.cpp
  SYCL :: spec_const/spec_const_redefine.cpp

Current tip commit: da6bfd0

However, I do not see them pass in the PR buildbot runs and try to figure out if to update to the newer driver, or not.

Are these expected with the mentioned driver version?
Are the buildbot machines using a different driver version?

@bader
Copy link
Contributor

bader commented Jun 11, 2020

I suppose these tests still fail on non-GPU devices.
The buildbot expects these tests to pass on all the configured devices.

Feel free to adjust the XFAIL pattern to fix status on your configuration.

@bjoernknafla
Copy link
Contributor Author

bjoernknafla commented Jun 11, 2020

@bader From what I see they are passing now for all OpenCL devices (though I do not have access to an FPGA for testing).

Do you want me to open a PR that removes the XFAILs - although I expect this to then fail the buildbots?

@bader
Copy link
Contributor

bader commented Jun 11, 2020

I was thinking about something like this:

// XFAIL: linux && opencl && !gpu && !cpu

@bjoernknafla
Copy link
Contributor Author

@bader That is a neat idea! PR with your idea is here: #1875

bader pushed a commit that referenced this issue Jul 22, 2020
The spec_const LIT tests start to pass for Intel CPU and GPU devices
with the Intel OpenCL GPU driver 20.19.16754.

Duplicate the test for accelerators, e.g., FPGA, and only fail it for
these.

As a result regressions are now caught for OpenCL CPU and GPU devices.

Fixes #1873

Signed-off-by: Bjoern Knafla <[email protected]>
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 a pull request may close this issue.

2 participants