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.

Sign up for GitHub

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

Jump to bottom

[Clang] constexpr support for __builtin_convertvector and __builtin_shufflevector #107985

Closed
philnik777 opened this issue Sep 10, 2024 · 4 comments · Fixed by #112129
Closed

[Clang] constexpr support for __builtin_convertvector and __builtin_shufflevector #107985

philnik777 opened this issue Sep 10, 2024 · 4 comments · Fixed by #112129
Labels
clang:frontend Language frontend issues, e.g. anything involving "Sema"

Comments

@philnik777
Copy link
Contributor

LanguageExtensions.rst claims that __builtin_shufflevector and __builtin_convertvector can be constant evaluated, but that's not reflected inside Butiltins.td.
@github-actions github-actions bot added the clang Clang issues not falling into any other category label Sep 10, 2024
@zwuis
Copy link
Contributor

zwuis commented Sep 10, 2024

This is part of #46593.

@RKSimon
Copy link
Collaborator

RKSimon commented Sep 22, 2024

This was fixed by #76615

@RKSimon RKSimon closed this as completed Sep 22, 2024
@EugeneZelenko EugeneZelenko added clang:frontend Language frontend issues, e.g. anything involving "Sema" and removed clang Clang issues not falling into any other category labels Sep 22, 2024
@llvmbot
Copy link
Member

llvmbot commented Sep 22, 2024

@llvm/issue-subscribers-clang-frontend

Author: Nikolas Klauser (philnik777)

LanguageExtensions.rst claims that `__builtin_shufflevector` and `__builtin_convertvector` can be constant evaluated, but that's not reflected inside `Butiltins.td`.

@philnik777
Copy link
Contributor Author

@RKSimon No, it hasn't: https://godbolt.org/z/c3bTe5Y8E. This report is about the inconsistency that it's documented as such inside LanguageExtensions.rst but not actually marked as such, i.e. a Constexpr attribute is missing from Builtins.td.

@philnik777 philnik777 reopened this Oct 6, 2024
@c8ef c8ef mentioned this issue Oct 13, 2024
Merged
c8ef added a commit that referenced this issue Oct 14, 2024
@c8ef
[clang] Mark __builtin_convertvector and __builtin_shufflevector …
ab6ec7a 
…as `constexpr`. (#112129)

Closes #107985.

LanguageExtensions.rst states that `__builtin_shufflevector` and
`__builtin_convertvector` can be evaluated as constants, but this is not
reflected in Butiltins.td. This patch aligns these two.
DanielCChen pushed a commit to DanielCChen/llvm-project that referenced this issue Oct 16, 2024
@c8ef @DanielCChen
[clang] Mark __builtin_convertvector and __builtin_shufflevector …
9d2ae5b 
…as `constexpr`. (llvm#112129)

Closes llvm#107985.

LanguageExtensions.rst states that `__builtin_shufflevector` and
`__builtin_convertvector` can be evaluated as constants, but this is not
reflected in Butiltins.td. This patch aligns these two.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
clang:frontend Language frontend issues, e.g. anything involving "Sema"
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[clang] Mark __builtin_convertvector and __builtin_shufflevector as constexpr. c8ef/llvm-project
5 participants
@RKSimon @EugeneZelenko @philnik777 @zwuis @llvmbot