bob80905 added a comment.
I think you need to mention this new builtin in clang/docs/ReleaseNotes.rst.
================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:2548
+ case Builtin::BI__builtin_sqrtf128:
+ case Builtin::BI__builtin_elementwise_sqrt: {
llvm::Value *Call = emitUnaryMaybeConstrainedFPBuiltin(
----------------
Nit: I think despite this code working, it should be moved to be grouped with
the other elementwise builtins at around line 3240.
Consider BI__builtin_log at around 2436 and the other log builtins, the
bultin_elementwise_log builtin is not in that group since it's with the other
elementwise builtins. Writing this case here would make an inconsistency with
the placement of the other elementwise builtins.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156737/new/
https://reviews.llvm.org/D156737
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
