[PATCH] D75788: [OpenMP] Provide math functions in OpenMP device code via OpenMP variants

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Apr 11 11:11:51 PDT 2020


jdoerfert marked an inline comment as done.
jdoerfert added inline comments.


================
Comment at: clang/test/Headers/nvptx_device_math_macro.cpp:11
+  double a(0);
+// CHECK-NOT:  call
+// CHECK:  call double @llvm.fabs.f64(double
----------------
sammccall wrote:
> Hmm, this fails if the test is run from a directory containing "call".
> 
> > ; ModuleID = '/usr/local/google/home/sammccall/src/llvm-project/clang/test/Headers/nvptx_device_math_macro.cpp
> 
> I guess I can blame my parents :-)
> Is `CHECK-NOT: call double` correct for both of these?
Oh my, ... sorry.

I guess `call{{.*}}@` should do the trick. No globals are referenced in a call


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75788





More information about the cfe-commits mailing list