Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[DOC] Major documentation update #200

Merged
merged 6 commits into from
Mar 31, 2021
Merged

[DOC] Major documentation update #200

merged 6 commits into from
Mar 31, 2021

Conversation

pvchupin
Copy link

  • Simplified instructions for the first local run
  • Clarified variables, parameters meaning
  • Fixed inconsistencies with implementation
  • Outlined special test categories External and ExtraTests
  • Other minor fixes

TODO:

  • Update LIT parameters, some are not used now, some are not documented

Signed-off-by: Pavel V Chupin [email protected]

* Simplified instructions for the first local run
* Clarified variables, parameters meaning
* Fixed inconsistencies with implementation
* Outlined special test categories External and ExtraTests
* Other minor fixes

TODO:
* Update LIT parameters, some are not used now, some are not documented

Signed-off-by: Pavel V Chupin <[email protected]>
@pvchupin pvchupin requested a review from vladimirlaz as a code owner March 26, 2021 06:51
vladimirlaz
vladimirlaz previously approved these changes Mar 26, 2021
vladimirlaz
vladimirlaz previously approved these changes Mar 26, 2021
@pvchupin pvchupin requested a review from bader March 26, 2021 18:26
Also found that -DCHECK_SYCL_ALL is really required for the first run
-DSYCL_TARGET_DEVICES alone is not working.

TODO: need to clarify External/ExtraTests functionality
bader
bader previously approved these changes Mar 30, 2021
@vladimirlaz vladimirlaz requested a review from bader March 31, 2021 05:55
@bader bader merged commit 4a204bd into intel:intel Mar 31, 2021
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
* Simplified instructions for the first local run
* Clarified variables, parameters meaning
* Fixed inconsistencies with implementation
* Outlined special test categories External and ExtraTests
* Other minor fixes

TODO:
* Update LIT parameters, some are not used now, some are not documented

Signed-off-by: Pavel V Chupin <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants