[PATCH] D137925: [AArch64] FNMUL

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

NoHonorSignDependentRounding is the old way of doing things - it should be based on fastmath flags nowadays. Or at least I've not heard of NoHonorSignDependentRounding before and it seems to be only used in the Arm backend at the moment.

This alive test seems to suggest that we can just always do the transform: https://alive2.llvm.org/ce/z/5-LdY1. Which I think sounds OK considering that it is a fneg. Do you know why HonorSignDependentRounding is needed? Was it just copied from Arm?


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

https://reviews.llvm.org/D137925