HomePhabricator

[OpenMP][NFCI] Introduce llvm/IR/OpenMPConstants.h

Authored by jdoerfert on Nov 4 2019, 8:00 PM.

Description

[OpenMP][NFCI] Introduce llvm/IR/OpenMPConstants.h

Summary:
The new OpenMPConstants.h is a location for all OpenMP related constants
(and helpers) to live.

This patch moves the directives there (the enum OpenMPDirectiveKind) and
rewires Clang to use the new location.

Initially part of D69785.

Reviewers: kiranchandramohan, ABataev, RaviNarayanaswamy, gtbercea, grokos, sdmitriev, JonChesterfield, hfinkel, fghanim

Subscribers: jholewinski, ppenzin, penzn, llvm-commits, cfe-commits, jfb, guansong, bollu, hiraditya, mgorny

Tags: #clang, #llvm

Differential Revision: https://reviews.llvm.org/D69853

Details

Committed
jdoerfertDec 9 2019, 10:10 PM
Differential Revision
D69853: [OpenMP][NFCI] Introduce llvm/IR/OpenMPConstants.h
Parents
rG444812500771: [BPF] Support to emit debugInfo for extern variables
Branches
Unknown
Tags
Unknown

Event Timeline

Hello Johannes,

I'm currently monitoring the ppc llvm buildbots and it seems that this patch causes a failure in one of our buildbots: clang-ppc64le-linux-multistage.

Here is the page of the build showing the failure: http://lab.llvm.org:8011/builders/clang-ppc64le-linux-multistage/builds/11325
Here is the output of the build: http://lab.llvm.org:8011/builders/clang-ppc64le-linux-multistage/builds/11325/steps/build%20stage%201/logs/stdio

Can you please commit a quick patch asap or revert the commit then investigate the failure ?

Thank you.

The issue is solved now and the buildbot is back to green. Thank you.