[PATCH] D82177: [flang][OpenMP] Enhance parser support for flush construct to OpenMP 5.0

Kiran Kumar T P via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 26 05:57:01 PDT 2020


kiranktp added a comment.

In D82177#2111512 <https://reviews.llvm.org/D82177#2111512>, @tskeith wrote:

> What is happening with this change? It looks like it was applied and then reverted.
>  @echristo, was there a problem?


Hi @echristo,
I am not facing any test failures with this patch on my sandbox. I tested this on multiple setups.
Below are my build details:

env CC=clang-8 CXX=clang++-8 CXXFLAGS="-stdlib=libc++" LDFLAGS="-fuse-ld=lld" cmake -GNinja -DCMAKE_BUILD_TYPE=Debug -DCMAKE_INSTALL_PREFIX=../install_clang -DLLVM_TARGETS_TO_BUILD=host -DLLVM_INSTALL_UTILS=on -DLLVM_ENABLE_PROJECTS="mlir;flang" ../../llvm
ninja -j8 install
ninja check-all

Let me know if I am missing anything.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82177





More information about the llvm-commits mailing list