[PATCH] D122589: Additionally set f32 mode with denormal-fp-math

Matt Arsenault via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 30 06:11:53 PDT 2022


arsenm added a comment.

I think my intent here was to not emit denormal-fp-math-f32 at all if it wasn't specified (i.e. denormal-fp-math-f32 is a special override only really used for AMDGPU)

The interaction between the two and the default is supposed to be handled by the backend interpretation of the attributes. Does this fix an observable problem for you?


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

https://reviews.llvm.org/D122589



More information about the cfe-commits mailing list