[PATCH] D142907: LangRef: Add "dynamic" option to "denormal-fp-math"

[Kevin P. Neal via Phabricator via llvm-commits]

kpn added inline comments.


================
Comment at: llvm/test/Transforms/Inline/AMDGPU/inline-denormal-fp-math.ll:78
+; CHECK-NEXT:    [[CALL:%.*]] = call i32 @func_default()
+; CHECK-NEXT:    ret i32 [[CALL]]
 ;
----------------
arsenm wrote:
> kpn wrote:
> > Are we changing the behavior in a way that may cause regressions? It looks like we've changed behavior in the absence of "dynamic".
> This case is broken to begin with, calling ieee from daz code. This makes the inlining more conservative / noticeable for debugging
Can I talk you into mentioning this in your commit message?


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

https://reviews.llvm.org/D142907