Varnike wrote:

Made fixes based on comments. Also corrected lowering of `STRICT_F16_TO_FP` and 
`STRICT_FP_TO_F16`.

>Yeah that might be useful - so we can add this part before turning the whole 
>feature on. There are some strict-fp vector operations that could 
>theoretically be supported - I'm not sure yet if they are very worth adding or 
>not.

For review convenience, I’ve left the changes here for now. If everything looks 
good, I’ll create a separate PR and revert this one to its original state.

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

Reply via email to