Skip to content

Conversation

@aelovikov-intel
Copy link
Contributor

Reverts #11155. I'm almost sure this is what broke our CI for the past few days.

@aelovikov-intel aelovikov-intel requested review from a team as code owners September 20, 2023 00:12
@aelovikov-intel aelovikov-intel merged commit b6b0cf4 into sycl Sep 20, 2023
@aelovikov-intel aelovikov-intel deleted the revert-11155-petr/upstream-ur-changes branch September 20, 2023 02:26
@ldrumm
Copy link
Contributor

ldrumm commented Sep 20, 2023

Who reviewed this?
"Almost sure" seems like a recipe for false confidence.

@veselypeta

@aelovikov-intel
Copy link
Contributor Author

"Almost sure" seems like a recipe for false confidence.

Pre-commit CI for this is the first one that passed in 4 days. I also reproduced fails/passes when doing manual reverts on the runner. Somehow, when running through CI most of those fails turn to hangs instead (that was the reason for "almost").

Who reviewed this?

I've merged it using my maintainers rights because we had to unblock CI ASAP and nobody else was willing to do anything with it.

veselypeta pushed a commit to veselypeta/llvm that referenced this pull request Sep 21, 2023
…l#11227)

Reverts intel#11155. I'm almost sure this is what broke our CI for
the past few days.
fabiomestre pushed a commit to fabiomestre/llvm that referenced this pull request Sep 26, 2023
…l#11227)

Reverts intel#11155. I'm almost sure this is what broke our CI for
the past few days.
veselypeta pushed a commit to veselypeta/llvm that referenced this pull request Sep 28, 2023
…l#11227)

Reverts intel#11155. I'm almost sure this is what broke our CI for
the past few days.
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.

3 participants