[PATCH] D127855: [OpenMP] Basic parse and sema support for modifiers in order clause

Sandeep via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 8 09:06:02 PST 2023


sandeepkosuri added a comment.

In D127855#4048642 <https://reviews.llvm.org/D127855#4048642>, @jyu2 wrote:

> In D127855#3956014 <https://reviews.llvm.org/D127855#3956014>, @sandeepkosuri wrote:
>
>> As I do not have commit access, can someone commit this patch, now that it passes the pre-merge tests ?
>
> I see some tests failed after this patch.  Failed only with -fopenmp-vesion=51
>
> https://www.godbolt.org/z/3oxWTcxn7

Hey @jyu2 , this error is not reproducible anymore, I think the issue is solved, by someone else's patch.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127855/new/

https://reviews.llvm.org/D127855



More information about the cfe-commits mailing list