[libc-commits] [clang] [libc] [llvm] [clang] Make __builtin_exp and __builtin_expf constexpr. (PR #199808)

[Manuel Carrasco via libc-commits]

mgcarrasco wrote:

Hi! This comment was generated using [fuzz-fill](https://github.com/ROCm/fuzz-fill), a new tool we're building to help improve LLVM test coverage by spotting lines introduced in a PR that don't yet appear to be exercised by the full test suite, including any tests added in this PR.

Looking at PR head 5993891614cf, we found 2 lines added or modified here that we couldn't find a test for:

1. https://github.com/llvm/llvm-project/blob/5993891614cfd1005824b7d53e2a8469a04f732a/clang/lib/AST/ByteCode/InterpBuiltin.cpp#L804
2. https://github.com/llvm/llvm-project/blob/5993891614cfd1005824b7d53e2a8469a04f732a/clang/lib/AST/ExprConstant.cpp#L20756

If it makes sense, adding a test that validates these lines would be great. And if any of them are already tested and we missed it, we'd love to hear so we can improve the tool. Feel free to give us some feedback on whether this review was useful from your perspective. Thanks!

https://github.com/llvm/llvm-project/pull/199808