MacDue wrote:

> LGTM, but like I mentioned on #107598, it would be good if there was a test 
> that requires the argument check, and the return check isn't sufficient

I've added a test case for `int ilogbl(long double a);` (which tests this in 
the `MINGW32` case :+1: 

https://github.com/llvm/llvm-project/pull/108853
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to