[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