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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
