[clang] [Clang][C++23] Implement P1774R8: Portable assumptions (PR #81014)

Alexey Bataev via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 8 10:43:07 PST 2024


alexey-bataev 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.

That would be great!

https://github.com/llvm/llvm-project/pull/81014


More information about the cfe-commits mailing list