This will be used in the opencl builtin headers to provide direct
intrinsic access with proper !fpmath metadata.
Details
Diff Detail
Event Timeline
I think you need to mention this new builtin in clang/docs/ReleaseNotes.rst.
clang/lib/CodeGen/CGBuiltin.cpp | ||
---|---|---|
2534 | Nit: I think despite this code working, it should be moved to be grouped with the other elementwise builtins at around line 3240. |
clang/lib/CodeGen/CGBuiltin.cpp | ||
---|---|---|
2534 | I think it's more important to group with the emission of the metadata Also don't understand why there is a broken constrained path and a redundant set of constrained handling in IRBuilder |
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.