================
@@ -342,6 +342,10 @@ template <typename ContextT> class 
GenericUniformityAnalysisImpl {
       typename SyncDependenceAnalysisT::DivergenceDescriptor;
   using BlockLabelMapT = typename SyncDependenceAnalysisT::BlockLabelMap;
 
+  // Use outside cycle with divergent exit
+  using UOCWDE =
----------------
ssahasra wrote:

Just a suggestion, I would consider giving the name "TemporalDivergenceList" to 
the entire type ``SmallVector<std::tuple<InstructionT* ...``, and then derive 
other names from the public member types exposed by the SmallVector. In 
particular, the function that returns this vector as a range can benefit from 
simply saying ``auto`` as its return types. This instance of "really long 
derived names" is justification enough for just saying ``auto`` instead. There 
is no value in repeating all these names.

https://github.com/llvm/llvm-project/pull/124298
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to