[PATCH] D134938: Use builtins in <math.h>
Nikolas Klauser via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 17 16:50:06 PDT 2022
philnik added a comment.
As @ldionne already mentioned, I've got a stack of patches to refactor `math.h`. I think the full stack should do everything you are trying to achieve here and more. I think you are just introducing dead code with the `__has_builtin()`s here. I've already got patches locally that change everything to builtins. Because of that I think this patch is redundant. Still, thanks for working on the patch. Do you want me to tag you on related patches I upload, so you can track development?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134938/new/
https://reviews.llvm.org/D134938
More information about the llvm-commits
mailing list