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

[Hans Wennborg via Phabricator via cfe-commits]

hans added inline comments.


================
Comment at: clang/test/OpenMP/amdgpu_exceptions.cpp:10
+
+// RUN: %clang_cc1 -fopenmp -triple amdgcn-amd-amdhsa -fopenmp-is-target-device -fcxx-exceptions -fexceptions %s -emit-llvm -S -verify=with -Wopenmp-target-exception -analyze
+// RUN: %clang_cc1 -fopenmp -triple amdgcn-amd-amdhsa -fopenmp-is-target-device -fcxx-exceptions -fexceptions %s -emit-llvm -S -verify=with -Wopenmp-target-exception -analyze
----------------
aeubanks wrote:
> I believe tests using `-analyze` need `REQUIRES: staticanalyzer`
Right, otherwise these fail in builds confiugured with -DCLANG_ENABLE_STATIC_ANALYZER=OFF.

Adding the requirement in 1968f0d7981df2d508c7c862d875b115837208b3.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153924