================
@@ -436,8 +482,8 @@ class CompilerType {
                      ExecutionContextScope *exe_scope);
 
   /// Dump to stdout.
-  void DumpTypeDescription(lldb::DescriptionLevel level =
-                           lldb::eDescriptionLevelFull) const;
----------------
felipepiovezan wrote:

I understand the desire to fix formatting issues, but let's avoid doing them in 
a big patch like this. If things needs to be reverted, as this patch was, such 
a formatting change should not need to be reverted.

This can also cause unnecessary work for downstream forks in case of merge 
conflicts (the first patch, the revert, the second patch, etc).

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

Reply via email to