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

[Sameer Sahasrabuddhe via Phabricator via llvm-commits]

sameerds added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/AttributorAttributes.cpp:8939
+      ModeF32 = Mode;
+    Known = DenormalState{Mode, ModeF32};
+  }
----------------
If all modes are non-dynamic, should this set optimistic fixed point?


================
Comment at: llvm/lib/Transforms/IPO/AttributorAttributes.cpp:8964
+
+    return Change;
+  }
----------------
If all modes are "not dynamic", should this call optimistic fixpoint?


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

https://reviews.llvm.org/D156129