[libcxx-commits] [libcxx] [libc++] Replace `__compressed_pair` with `[[no_unique_address]]` (PR #76756)

Jonathan Wakely via libcxx-commits libcxx-commits at lists.llvm.org
Wed Dec 4 07:16:38 PST 2024


================
@@ -231,7 +199,7 @@ def __init__(self, val):
 
     def to_string(self):
         """Build a python string from the data whether stored inline or separately."""
-        value_field = _value_of_pair_first(self.val["__r_"])
+        value_field = self.val["__rep_"]
----------------
jwakely wrote:

I suppose if you're manually loading the printers and calling the registration function, yes you could do that (and switch back and forth if the application contains both definitions). You would need to have both `printer.py` files on disk, in known locations. If you don't have the LLVM source tree present, I'm not sure you'd have the old `printers.py` to load it.

For libstdc++ the printers are loaded automatically when `libstdc++.so.6` is part of the process, so the user doesn't source them manually or register the printers manually. We want it to Just Work.

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


More information about the libcxx-commits mailing list