teemperor accepted this revision.
teemperor added a comment.
This revision is now accepted and ready to land.

I think this can be further simplified, but we can also do the move to 
`QualType::getDesugaredType` in a follow-up commit to unblock your atomic 
struct patch.



================
Comment at: lldb/source/Symbol/ClangASTContext.cpp:2489
+    case clang::Type::Typedef:
+      type = cast<clang::TypedefType>(type)->getDecl()->getUnderlyingType();
+      break;
----------------
QualType already has a `getDesugaredType` implementation that does the same 
(but handles more cases, so you still need the switch statement but you can 
unify all the case bodies if you want to keep this NFC).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71212



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to