This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Initial parsing/sema for the 'omp parallel loop' construct
ClosedPublic

Authored by mikerice on Mar 22 2022, 11:35 AM.

Details

Reviewers
ABataev
jdoerfert
Summary

Adds basic parsing/sema/serialization support for the #pragma omp parallel loop directive.

Diff Detail

Event Timeline

mikerice created this revision.Mar 22 2022, 11:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 22 2022, 11:35 AM
mikerice requested review of this revision.Mar 22 2022, 11:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 22 2022, 11:35 AM
This revision is now accepted and ready to land.Mar 22 2022, 12:22 PM
mikerice closed this revision.Mar 23 2022, 12:02 PM

Hi @Jake-Egan, not sure I understand. This change was checked in 22 Mar 13:55:47 but the changes tested in this build are from 23 Mar.
Is the test still failing?

Hi @Jake-Egan, not sure I understand. This change was checked in 22 Mar 13:55:47 but the changes tested in this build are from 23 Mar.
Is the test still failing?

Sorry, I sent you a link to a newer build. The test first started failing here: https://lab.llvm.org/buildbot/#/builders/214/builds/365/steps/6/logs/FAIL__Clang__index-source_m

I double-checked to make sure this commit caused the failure.. but it looks like the test no longer fails on the latest builds, so I think we're good.

martong removed a subscriber: martong.May 5 2022, 2:51 AM