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.

Sign up for GitHub

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

Jump to bottom

Add catching and report errors to the testgen run script #740

Closed
wants to merge 354 commits into from
Closed

Add catching and report errors to the testgen run script #740

wants to merge 354 commits into from

Conversation

Garra1980
Copy link
Contributor

Please review these guidelines to help with the review process:

  • Have you provided a meaningful PR description?
  • Have you added a test, a reproducer, or a reference to an issue with a reproducer?
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?
  • If this PR is a work in progress, are you filing the PR as a draft?
  • Have you organized your commits logically and ensured each can be built by itself?

MLIR Extensions and others added 30 commits April 5, 2023 16:45
Merge remote-tracking branch 'upstream/main' into main
6856e91
@XinyueLunaFan
Update SDL user
720ed10
@XinyueLunaFan
Update sdl.yml
cf68546
Merge remote-tracking branch 'upstream/main' into main
85cd6aa
@XinyueLunaFan
Update SDL Credential
8ec4ff4
@mshahneo
Add BF16 GEMM test case (#62)
31eebef 
It does GEMM on two BF16 matrices and Accumulates in F32 matrix
Merge remote-tracking branch 'upstream/main' into main
547c393
@XinyueLunaFan
Enable CI on dist-ndarray CPU build
8211f38
@XinyueLunaFan
Enable CI on dist-ndarray GPU build
c3fa4ff
@XinyueLunaFan
Exclude unwanted folders from SDL scan
9a68a1f
Merge remote-tracking branch 'upstream/main' into main
b172bb6
Add requirements.txt file
445f5b8
@XinyueLunaFan
Update requirements.txt
d930761
@silee2
Fix pre-commit issues for innersource only files. (#68)
08cf84c
Merge remote-tracking branch 'upstream/main' into main
bda2a0a
@XinyueLunaFan
Clear requirements.txt
1c75fd6
@XinyueLunaFan
Update sdl.yml
52a37ae
Merge remote-tracking branch 'upstream/main' into main
256e333
Merge remote-tracking branch 'upstream/main' into main
33db568
@XinyueLunaFan
Update sdl.yml
6c1679b
@silee2
Add set -o pipefail before using pipe for capturing test output. (#77)
c8d6a24
@drprajap
SDL Process - Remove Snyk and coverity exclude files command-line
e046b11
Merge remote-tracking branch 'upstream/main' into main
6659a04
@silee2
Fix incorrect level zero install path.
65ddb96 
Source oneapi before configuring IMEX.
Remove redundant SYCL_DIR.
@silee2
Merge remote-tracking branch 'upstream/main'
86fba46
@XinyueLunaFan
Merge remote-tracking branch 'upstream/main' into main
459fb65
@silee2
Simplify CPU and GPU CI
c1f30d1 
Assumptions:

Github actions is serialized. (Running one action at a time)
CI uses system installed cmake (not from anaconda)

List of changes:

Remove usage of anaconda
Does not cache LLVM build
Local persistent LLVM source repo is shared between cpu and gpu CI. We can do this since CI actions are serialized. cpu and gpu CI actions will pull upstream change an checkout LLVM commit id based on PR.
LLVM/MLIR is always built from scratch
IMEX is built along LLVM as an external project instead of a separate build. This setup cuts LLVM build time since it reduce the number of targets need to be built for LLVM.
Single cmake target "check-imex" is invoked. This target both builds IMEX and runs test.
@silee2
Merge remote-tracking branch 'upstream/main'
4c94ae0
@silee2
Merge remote-tracking branch 'upstream/main' into multiauthor
48c1c6c
@drprajap
Create test.yml
faea7ba
Garra1980 and others added 28 commits April 24, 2024 14:59
@Garra1980
add lit restrictions to cpu builds also
db94d80
@Garra1980
Remove timeout for lit tests
21dafee
@Garra1980
Add launching sg-level xetile generated tests (#431)
a7340be
@Garra1980
Move CPU CI launches on the runner without PVC (#432)
48d6723 
* Update build_cpu.yml
@nbpatel
Add check for order attribute in Xetile.init_tile verifier (#730)
fef3160
@leshikus
Merge remote-tracking branch 'upstream/main'
ba30be7
@mshahneo
Merge remote-tracking branch 'upstream/main'
9539371
@chencha3
Merge remote-tracking branch 'upstream/main'
0b3015c
@chencha3
Merge remote-tracking branch 'upstream/main'
9ea13c2
@drprajap
Merge remote-tracking branch 'upstream/main'
e7cf799
@chencha3
Merge remote-tracking branch 'upstream/main'
c8dde7a
@chencha3
Merge remote-tracking branch 'upstream/main'
7ba1967
@leshikus
Merge remote-tracking branch 'upstream/main'
0a60ab8
@Garra1980
Adding check for incorrect test execution
610bdb0
@Garra1980
Update run_tests.sh
0a8db3a
@Garra1980
Update run_tests.sh
eb0c723
@Garra1980
Update run_tests.sh
1cc4b72
@Garra1980
Update run_tests.sh
faf6642
@Garra1980
Update run_tests.sh
ae4fb8e
@Garra1980
Dummy change to break gen tests
4c826a5
@Garra1980
a bit different way of "bad" test gen
2712fff
@Garra1980
Update run_tests.sh
99ee73e
@Garra1980
Update run_tests.sh
48d1215
@Garra1980
Use set pipefail
26d2290
@Garra1980
Update xetile_testgen.py
ca1b85f
@Garra1980
don't stop in case of errors
6b61896
@Garra1980
Trigger an error
bc0da6e
@Garra1980
Restore test gen
9aeaa9c
@Garra1980 Garra1980 closed this May 10, 2024
Garra1980 pushed a commit that referenced this pull request Oct 10, 2024
@silee2
CL/sycl.hpp is deprecated, use sycl/sycl.hpp (#740)
1b4d9a7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.