[PATCH] D21704: [PM] Port float2int to the new pass manager

[Davide Italiano via llvm-commits]

davide added inline comments.

================
Comment at: lib/Transforms/Scalar/Float2Int.cpp:73
@@ -88,1 +72,3 @@
+                      false)
 INITIALIZE_PASS_DEPENDENCY(GlobalsAAWrapperPass)
+INITIALIZE_PASS_END(Float2IntLegacyPass, "float2int", "Float to int", false,
----------------
This is not your bug, but still worth mentioning. This pass preserves globalsAA, doesn't depend on it. In other words, this can go away.

================
Comment at: lib/Transforms/Scalar/Float2Int.cpp:537
@@ +536,3 @@
+PreservedAnalyses Float2IntPass::run(Function &F, FunctionAnalysisManager &) {
+  //FIXME: skipFunction is not currently supported in the new PM.
+  if (!Impl.runImpl(F))
----------------
davidxl wrote:
> This is a good point. Is there a bug tracking this missing functionality?
I'd remove the FIXME. No other pass mentions it, and I'm not sure if there's active work on this.

================
Comment at: lib/Transforms/Scalar/Float2Int.cpp:541
@@ +540,3 @@
+  else {
+    //FIXME: setPreservesCFG is not currently supported in the new PM.
+    PreservedAnalyses PA;
----------------
davidxl wrote:
> same here.
I don't think there's a bug tracker for this (probably there should be one tho?). 
As a general comment, this needs an understanding which passes actually now 'preserve the CFG' (whatever that means, we need to agree on a sane/clear semantic) so I'm not entirely sure the changes here will be NFC. 


http://reviews.llvm.org/D21704