pengfei added a comment.

Yes, it is using maxpd/minpd here. Sorry for missing the nan cases.
In fact, I doubt about the behavior when using fast math with target 
intrinsics. In my opinion, target intrinsics are always associated with given 
instructions (reduce* are exceptions). So the behavior of intrinsics, e.g. 
respect nans, signaling, rounding, exception etc. are concordance with their 
associated instructions. That said the fast math flags won't change the 
behavior of intrinsics.
Assume above, I'm happy to set these expansions to fast math. Either keeping 
the existing implementaion or expansion LGTM.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93179/new/

https://reviews.llvm.org/D93179

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to