[PATCH] D63844: [LoopFusion] Extend use of OptimizationRemarkEmitter

[Kit Barton via Phabricator via llvm-commits]

kbarton marked an inline comment as done.
kbarton added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopFuse.cpp:1173
+             << Stat.getDesc());
+  }
 };
----------------
jdoerfert wrote:
> I would somehow prefer if the two functions above were merged or at least one uses the other. The ORE part is almost identical and we could, probably should, increment the relevant statistic in `reportLoopFusion` as well.
I agree passing the stat into reportLoopFusion makes sense. I'll make that change.

I don't quite understand the point about commoning these two functions into a single function though. They fundamentally do two different things: for successful fusion we emit to OptimizationRemark; for unsuccessful fusion we emit to OptimizationRemarkMissed.

So, in order to common these up there needs to be a check to figure out which OptimizationRemark class to use. At that point, the only commonality that you get is the increment of Stat. 

Am I missing something?



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63844