[PATCH] D145238: [NVPTX] Expose LDU builtins

[Jakub Chlanda via Phabricator via llvm-commits]

jchlanda added inline comments.


================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:18267-18271
+    auto HalfSupport = HasHalfSupport(BuiltinID);
+    if (!HalfSupport.first) {
+      CGM.Error(E->getExprLoc(),
+                HalfSupport.second.append(" requires native half type support.")
+                    .c_str());
----------------
tra wrote:
> I think we can simplify it all further.
> 
> ```
> auto HasHalfSupport = [&](unsigned BuiltinID) {
>     auto &Context = getContext();
>     return Context.getLangOpts().NativeHalfType || !Context.getTargetInfo().useFP16ConversionIntrinsics();
> }
> ...
> 
> if (!HasHalfSupport(BuiltinID)) {
>       CGM.Error(E->getExprLoc(),   getContext().BuiltinInfo.getName(BuiltinID) + " requires native half type support.");
> 
> ```
Done, although we need a string append there, StringRef and Twine would not work.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145238