[Lldb-commits] [PATCH] D116217: [lldb] Fix PR52702 by fixing Mangled::operator!

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Dec 28 05:41:37 PST 2021


labath added inline comments.


================
Comment at: lldb/source/Core/Mangled.cpp:87
 //  { ...
-bool Mangled::operator!() const { return !m_mangled; }
+bool Mangled::operator!() const { return !m_mangled && !m_demangled; }
 
----------------
rZhBoYao wrote:
> Should we remove `Mangled::operator!` altogether? I find it less confusing without. 
Does `!mangled` work without it? Would it be sufficient to implement the `!` operator in terms of bool conversion function?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116217/new/

https://reviews.llvm.org/D116217



More information about the lldb-commits mailing list