Skip to content

A workaround for LLVM bug, https://github.com/llvm/llvm-project/issues/58633#43195

Merged
davenger merged 1 commit intomasterfrom
workaround-for-llvm-bug
Nov 14, 2022
Merged

A workaround for LLVM bug, https://github.com/llvm/llvm-project/issues/58633#43195
davenger merged 1 commit intomasterfrom
workaround-for-llvm-bug

Conversation

@alexey-milovidov
Copy link
Copy Markdown
Member

@alexey-milovidov alexey-milovidov commented Nov 13, 2022

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

llvm/llvm-project#58633

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Nov 13, 2022
@davenger davenger self-assigned this Nov 13, 2022
@alexey-milovidov
Copy link
Copy Markdown
Member Author

alexey-milovidov commented Nov 13, 2022

@tavplubix a test has failed: 02449_check_dependencies_and_table_shutdown
https://s3.amazonaws.com/clickhouse-test-reports/43195/f5270a6ebeeac6221079524b4a602908e26d6f0c/stateless_tests__debug__[3/3].html

@alexey-milovidov
Copy link
Copy Markdown
Member Author

alexey-milovidov commented Nov 13, 2022

Integration tests (tsan) [2/4] — fail: 3, passed: 541, flaky: 3

@antonio2368 said that the test "test_keeper_four_word_command" will be fixed in #43201.

@davenger davenger merged commit 992729f into master Nov 14, 2022
@davenger davenger deleted the workaround-for-llvm-bug branch November 14, 2022 12:46
@itrofimow itrofimow mentioned this pull request Nov 15, 2022
Closed
azat added a commit to azat/ClickHouse that referenced this pull request May 8, 2023
@azat
Revert "Merge pull request ClickHouse#43195 from ClickHouse/workaroun…
e278081 
…d-for-llvm-bug"

With clang 16 this workaround should not be required anymore.

This reverts commit 992729f, reversing
changes made to d885b3c.
@azat azat mentioned this pull request May 8, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@davenger davenger davenger approved these changes

Assignees

@davenger davenger

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alexey-milovidov @davenger @robot-clickhouse