[PATCH] D91980: [OpenMP] Add initial support for `omp [begin/end] assumes`

Wed Dec 16 05:34:06 PST 2020

thakis added a comment.

In D91980#2456637 <https://reviews.llvm.org/D91980#2456637>, @jdoerfert wrote:

> In D91980#2456600 <https://reviews.llvm.org/D91980#2456600>, @thakis wrote:
>
>> In addition to the build error, it also broke check-clang everywhere as far as I can tell.
>> mac: http://45.33.8.238/macm1/292/step_6.txt
>> linux: http://45.33.8.238/linux/35399/step_7.txt
>> win: http://45.33.8.238/win/29934/step_7.txt
>
> I think I fixed 2 errors but I'm unsure about the linux one. I changed one test and I'll try it again as I find a workaround for gcc-5

The same tests fail on linux and mac and win at these links:

  Clang :: OpenMP/assumes_codegen.cpp (win, mac/m1)
  Clang :: OpenMP/assumes_include_nvptx.cpp (linux, mac/x86, mac/m1)

(mac/x86 link: http://45.33.8.238/mac/25261/step_7.txt)

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91980