[clang] [clang][OpenMP] Add error for large expr in collapse clause (PR #138592)

Amr Hesham via cfe-commits cfe-commits at lists.llvm.org
Fri May 9 13:12:59 PDT 2025


AmrDeveloper wrote:

> Thank you for the fix! I've added a few more reviewers. This is a pretty general problem, so I think we may want additional test coverage for basically any of the OpenMP clauses which accept an integer argument. For example, this is another related issue: #139268 -- I expect they're all resolved by your fix, perhaps?

Thank you for adding reviewers.

It will not fix the other issue because `ActOnOpenMPUnrollDirective` uses `getZValue` directly and doesn't call `VerifyPositiveIntegerConstantInClause`, the solution will be similar but inside `ActOnOpenMPUnrollDirective`

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


More information about the cfe-commits mailing list