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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
