hokein accepted this revision.
hokein added inline comments.
This revision is now accepted and ready to land.
================
Comment at: clang/lib/Sema/SemaCodeComplete.cpp:5173
+ if (auto *PLE = llvm::dyn_cast<ParenListExpr>(Base))
+ Base = PLE->getExpr(PLE->getNumExprs() - 1);
+ if (OtherOpBase) {
----------------
mention that why we specialize `ParentListExpr` -- ParenListExprs don't have a
predefined type
I think we won't (hopefully) have a 0-children `ParenListExpr`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95935/new/
https://reviews.llvm.org/D95935
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
