AaronBallman wrote:

> > the actual high-level goal here (from my "i'm just the libc maintainer" 
> > perspective) was "clang should automatically replace calls to isnanf() with 
> > an instruction or two, like it already does with isnan()".
> 
> If we want the compiler to automatically recognize calls to libc isnanf, 
> without a define in libc, implementing __builtin_isnanf doesn't help; we need 
> to recognize calls to libc isnanf (either in clang, or in LLVM 
> SimplifyLibCalls).

Also, if we're auto recognizing `isnanf` as a library builtin, should we handle 
`_isnanf` too 
(https://learn.microsoft.com/en-us/previous-versions/tzthab44(v=vs.140))?

https://github.com/llvm/llvm-project/pull/218004
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to