[clang] [Clang][C++23] Implement P1774R8: Portable assumptions (PR #81014)
via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 8 10:40:54 PST 2024
Sirraide wrote:
> > > > `[[clang::assume]]` or `__attribute__((assume))` are the OpenMP one
> > >
> > >
> > > OpenMP requires only [[omp::assume]], neither [[clang::assume]] nor **attribute**((assume)) are OpenMP requirement.
> >
> >
> > Yeah, the weird thing is I’m not sure we support `[[omp::assume]]` at the moment. We should definitely fix this entire situation, but I’m not sure it needs to be part of this pr.
>
> Can you somehow fix omp::assume at first? Say, in a separate patch?
That’s independent of the changes introduced by this pr, from what I can tell at least. So it should be possible to do that irrespective of when this gets merged, yeah.
https://github.com/llvm/llvm-project/pull/81014
More information about the cfe-commits
mailing list