[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