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

[SYCL] Split optimization_level_debug_info test #977

Merged

Conversation

steffenlarsen
Copy link

The optimization_level_debug_info test can on some systems cause a timeout due to the compilation time without optimizations and with debug symbols enabled. To avoid this timeout, this commit splits the test into two parts; one checking with integral optimization levels (e.g. -O2) and one checking with special-name optimization levels (e.g. -Og).

The optimization_level_debug_info test can on some systems cause a
timeout due to the compilation time without optimizations and with
debug symbols enabled. To avoid this timeout, this commit splits the
test into two parts; one checking with integral optimization levels
(e.g. -O2) and one checking with special-name optimization levels
(e.g. -Og).

Signed-off-by: Steffen Larsen <[email protected]>
@steffenlarsen steffenlarsen requested a review from a team as a code owner April 6, 2022 13:21
Copy link

@cperkinsintel cperkinsintel left a comment

Choose a reason for hiding this comment

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

LGTM

@cperkinsintel
Copy link

this PR ( #983 ) temporarily disables the test that is failing here. Under investigation.

@steffenlarsen steffenlarsen merged commit f7b4cb1 into intel:intel Apr 19, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Jun 17, 2022
The optimization_level_debug_info test can on some systems cause a
timeout due to the compilation time without optimizations and with
debug symbols enabled. To avoid this timeout, this commit splits the
test into two parts; one checking with integral optimization levels
(e.g. -O2) and one checking with special-name optimization levels
(e.g. -Og).

Signed-off-by: Steffen Larsen <[email protected]>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…e#977)

The optimization_level_debug_info test can on some systems cause a
timeout due to the compilation time without optimizations and with
debug symbols enabled. To avoid this timeout, this commit splits the
test into two parts; one checking with integral optimization levels
(e.g. -O2) and one checking with special-name optimization levels
(e.g. -Og).

Signed-off-by: Steffen Larsen <[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.

2 participants