[PATCH] D156129: Attributor: Try to propagate concrete denormal-fp-math{-f32}

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/include/llvm/Transforms/IPO/Attributor.h:5063-5066
+      if (Caller.ModeF32 == DenormalMode::getInvalid())
+        Caller.ModeF32 = Caller.Mode;
+      if (Callee.ModeF32 == DenormalMode::getInvalid())
+        Callee.ModeF32 = Callee.Mode;
----------------
It turns out moving this to handle the invalid mode at initialize broke the partially dynamic mode case from a fixed caller (and my test for that was accidentally dead and deleted). At this point I'll fix that separately if it matters


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

https://reviews.llvm.org/D156129