[PATCH] D50433: A New Divergence Analysis for LLVM

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: include/llvm/Analysis/KernelDivergenceAnalysis.h:1
-//===- llvm/Analysis/DivergenceAnalysis.h - Divergence Analysis -*- C++ -*-===//
+//===- llvm/Analysis/KernelDivergenceAnalysis.h - KernelDivergence Analysis -*- C++ -*-===//
 //
----------------
arsenm wrote:
> I don't like the use of the name kernel here. This has nothing to do with kernels, and works fine for non-kernel functions (ignoring the flaws with the pass0
Better option might just be DivergenceAnalysisLegacy?


Repository:
  rL LLVM

https://reviews.llvm.org/D50433