[PATCH] D141355: [AMDGPU] Add NewPM support to AMDGPUUnifyDivergentExitNodes pass
Thomas Symalla via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Jan 22 23:57:53 PST 2023
tsymalla added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUUnifyDivergentExitNodes.cpp:97
/// XXX - Is there a more efficient way to find this?
-static bool isUniformlyReached(const LegacyDivergenceAnalysis &DA,
- BasicBlock &BB) {
+template <typename T> static bool isUniformlyReached(T *DA, BasicBlock &BB) {
SmallVector<BasicBlock *, 8> Stack(predecessors(&BB));
----------------
Can you give a more meaningful name to the template argument? For instance, `DivergenceAnalysisType`?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141355/new/
https://reviews.llvm.org/D141355
More information about the llvm-commits
mailing list