[PATCH] D75889: AMDGPU: Use V_MAC_F32 for fmad.ftz

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 10 11:28:18 PDT 2020


arsenm added a comment.

In D75889#1915361 <https://reviews.llvm.org/D75889#1915361>, @rampitec wrote:

> It was used in the div expansion for some non obvious reason. As far as I know it means 0.0 * x = 0.0, but I am not sure it is needed in this case.
>  Please run PSDB on this change, if it passes then LGTM.


It’s used in the div expansion because we want it regardless of the denormal mode. My confusion is for the f16 case, where the instruction was renamed or changed for some reason


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

https://reviews.llvm.org/D75889





More information about the llvm-commits mailing list