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.

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

clang-tidy reports: unknown pragma ignored #9165

Closed
JinheLin opened this issue Jun 27, 2024 · 1 comment · Fixed by #9166
Closed

clang-tidy reports: unknown pragma ignored #9165

JinheLin opened this issue Jun 27, 2024 · 1 comment · Fixed by #9166
Labels
severity/moderate type/bug The issue is confirmed as a bug.

Comments

@JinheLin
Copy link
Contributor


/home/jenkins/agent/workspace/tiflash-build-common/tiflash/dbms/src/Common/TargetSpecific.h:559:1: error: unknown pragma ignored [clang-diagnostic-unknown-pragmas]

  559 | TIFLASH_TARGET_SPECIFIC_NAMESPACE(template <size_t LENGTH> struct Word {

      | ^

/home/jenkins/agent/workspace/tiflash-build-common/tiflash/dbms/src/Common/TargetSpecific.h:535:5: note: expanded from macro 'TIFLASH_TARGET_SPECIFIC_NAMESPACE'

  535 |     TIFLASH_AVX_NAMESPACE(__VA_ARGS__)         \

      |     ^

/home/jenkins/agent/workspace/tiflash-build-common/tiflash/dbms/src/Common/TargetSpecific.h:494:5: note: expanded from macro 'TIFLASH_AVX_NAMESPACE'

  494 |     TIFLASH_BEGIN_AVX_SPECIFIC_CODE           \

      |     ^

/home/jenkins/agent/workspace/tiflash-build-common/tiflash/dbms/src/Common/TargetSpecific.h:54:9: note: expanded from macro 'TIFLASH_BEGIN_AVX_SPECIFIC_CODE'

   54 |         _Pragma("clang attribute push(__attribute__((target(\"sse,sse2,sse3,ssse3,sse4,popcnt,avx,avx2\"))),apply_to=function)")

      |         ^

note: expanded from here
@JinheLin JinheLin added the type/bug The issue is confirmed as a bug. label Jun 27, 2024
@JinheLin
Copy link
Contributor Author

JinheLin commented Jun 27, 2024

This is a bug in LLVM 17 when compiling for C++20.
Should be fixed in 18.
See llvm/llvm-project#67380.

ref #7193

ti-chi-bot bot pushed a commit that referenced this issue Jun 27, 2024
close #9165

Co-authored-by: JaySon <tshent@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity/moderate type/bug The issue is confirmed as a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant