https://github.com/jdoerfert commented:
Generally, getting rid of the pair is great.
I am unsure I understand why we do the base template rather than inheritance.
Where is the base template used? If we do inheritance we could avoid
duplicating of members and provide default impls that work with
https://github.com/bwendling updated
https://github.com/llvm/llvm-project/pull/76882
>From ca7a96a40952fe94b916dacc52f07aa90bbdb1e7 Mon Sep 17 00:00:00 2001
From: Bill Wendling
Date: Wed, 3 Jan 2024 13:22:37 -0800
Subject: [PATCH 1/3] [builtin_object_size] Use classes instead of std::pair
(NFC