[llvm] [Support][ADT] Minor cleanups after #101706 (PR #102180)

Jakub Kuderski via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 6 10:03:06 PDT 2024


================
@@ -262,7 +262,9 @@ class DominatorTreeBase {
       SmallVector<std::unique_ptr<DomTreeNodeBase<NodeT>>>;
   DomTreeNodeStorageTy DomTreeNodes;
   // For graphs where blocks don't have numbers, create a numbering here.
-  DenseMap<const NodeT *, unsigned> NodeNumberMap;
+  std::conditional_t<!GraphHasNodeNumbers<NodeT *>,
+                     DenseMap<const NodeT *, unsigned>, std::tuple<>>
----------------
kuhar wrote:

nit: consider adding todo to switch to an empty struct with `no_unique_address ` once we update to C++20.

https://github.com/llvm/llvm-project/pull/102180


More information about the llvm-commits mailing list