farzonl wrote:

> Right, asinf doesn't exist on Windows (see 
> llvm/lib/Analysis/TargetLibraryInfo.cpp). We probably need code to promote 
> calls to llvm.asin.f32 to 64-bit. Not sure how hard it would be to revert 
> this... probably easier to fix forward if we can do that quickly.
> 
> We'll need a fix for the regression for the 19 branch.

I see that  promoting seems like it could work, but it is also disabled for 
f80\f128:
https://github.com/llvm/llvm-project/blob/39e192b379362e9e645427631c35450d55ed517d/llvm/lib/Analysis/TargetLibraryInfo.cpp#L322C5-L324C39

How should we handle that case?

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

Reply via email to