[PATCH] D149588: clang: Start emitting intrinsic for __builtin_ldexp*

[Artem Belevich via Phabricator via llvm-commits]

tra added a comment.

Can you add a test for CUDA/NVPTX, too?



================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:3057-3058
+  case Builtin::BI__builtin_ldexpf:
+  case Builtin::BI__builtin_ldexpl:
+  case Builtin::BI__builtin_ldexpf16: {
+    return RValue::get(emitBinaryExpMaybeConstrainedFPBuiltin(
----------------
Should those builtins that map to an experimental intrinsic be `experimental`, too?

Alternatively, if the intrinsic functionality is something we intend to officially provide via clang builtin, should we drop the `experimental` part?



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

https://reviews.llvm.org/D149588