[Lldb-commits] [lldb] [lldb] Start using formatv() in RegisterContextUnwind (NFCI) (PR #191576)

[David Spickett via lldb-commits]

================
@@ -60,6 +60,16 @@ static bool CallFrameAddressIsValid(ABISP abi_sp, lldb::addr_t cfa) {
   return cfa != 0 && cfa != 1;
 }
 
+#define UNWIND_LOG(log, ...)                                                   \
+  LLDB_LOG(log, "{0}th{1}/fr{2} {3}",                                          \
+           llvm::indent(std::min(m_frame_number, 100U)),                       \
+           m_thread.GetIndexID(), m_frame_number, llvm::formatv(__VA_ARGS__))
+
+#define UNWIND_LOG_VERBOSE(log, ...)                                           \
+  LLDB_LOG_VERBOSE(                                                            \
+      log, "{0}th{1}/fr{2} {3}", llvm::indent(std::min(m_frame_number, 100U)), \
+      m_thread.GetIndexID(), m_frame_number, llvm::formatv(__VA_ARGS__))
----------------
DavidSpickett wrote:

I suppose it depends which is more potentially confusing, one more level of macro or one more copy of the same text.

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