[PATCH] D49491: [RFC][VPlan, SLP] Add simple SLP analysis on top of VPlan.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: lib/Transforms/Vectorize/VPlan.h:1387
+  /// A DenseMapInfo implementation for holding DenseMaps and DenseSets of
+  /// sorted SmallVectors of const SCEV*.
+  struct BundleDenseMapInfo {
----------------
fhahn wrote:
> dcaballe wrote:
> > SCEV -> VPValue?
> > 
> > I guess we couldn't have a single templatized version for sorted SmallVectors of any pointer, right?
> I think it is possible to provide a single implementation for SmallVectors. I've removed the implementation here and will upload a separate patch for it soon.
I've added a patch implementing DenseMap key info for smallvector: D52312


https://reviews.llvm.org/D49491