[flang] [llvm] [FLANG][OpenMP]Add support for ALIGN clause on OMP ALLOCATE (PR #120791)

Mats Petersson via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 3 10:43:39 PST 2025


Leporacanthicus wrote:

> Could you add a semantic check that the value is a compile-time positive constant, and some tests for it?

Done. Took a bit longer than I wanted, because I didn't realize my test-case was actually not going where it SHOULD go, so I added the executable variant, so it catches both types. And I had to add some more test-code to hit the declarative align side too. I guess whoever did the executable version never covered the align (or did I add that too?)

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


More information about the llvm-commits mailing list