[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