[PATCH] D153924: [OpenMP] Allow exceptions in target regions when offloading to GPUs

[Johannes Doerfert via Phabricator via cfe-commits]

jdoerfert added a comment.

I think this is basically done, one question though.



================
Comment at: clang/test/OpenMP/amdgpu_exceptions.cpp:11
+// RUN: %clang_cc1 -fopenmp -triple amdgcn-amd-amdhsa -fopenmp-is-target-device -fexceptions %s -emit-llvm -S -verify=with -Wopenmp-target-exception -o - &> /dev/null
+// RUN: %clang_cc1 -fopenmp -triple amdgcn-amd-amdhsa -fopenmp-is-target-device %s -emit-llvm -S -verify=with -Wopenmp-target-exception -o - &> /dev/null
+
----------------
Can we use /dev/null? Do other tests use it? I would expect -analyze or sth instead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153924