[PATCH] D82805: [clang] Fix a crash on passing C structure of incompatible type to function with reference parameter

Bruno Ricci via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 29 14:59:05 PDT 2020


riccibruno added inline comments.


================
Comment at: clang/lib/Sema/SemaInit.cpp:4539
+    if (CXXRecordDecl *T2RecordDecl =
+        dyn_cast<CXXRecordDecl>(T2RecordType->getDecl())) {
+      const auto &Conversions = T2RecordDecl->getVisibleConversionFunctions();
----------------
I cannot reproduce this with the provided test case. Can you explain why `T2RecordDecl` would not be a `CXXRecordDecl` since we should only get there with c++?


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

https://reviews.llvm.org/D82805





More information about the cfe-commits mailing list