[PATCH] D148474: [Clang] Fix ResolveConstructorOverload to not select a conversion function if we are going use copy elision

Shafik Yaghmour via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Apr 16 12:01:20 PDT 2023


shafik created this revision.
shafik added reviewers: rsmith, aaron.ballman, erichkeane, dim.
Herald added a project: All.
shafik requested review of this revision.

`ResolveConstructorOverload` needs to check properly if we are going to use copy elision we can't use a conversion function.

This fixes:

https://github.com/llvm/llvm-project/issues/39319
https://github.com/llvm/llvm-project/issues/60182
https://github.com/llvm/llvm-project/issues/62157

      


https://reviews.llvm.org/D148474

Files:
  clang/lib/Sema/SemaInit.cpp
  clang/lib/Sema/SemaTemplateInstantiate.cpp
  clang/test/SemaCXX/cxx1z-copy-omission.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D148474.514041.patch
Type: text/x-patch
Size: 7399 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230416/9f4a4ce3/attachment.bin>


More information about the cfe-commits mailing list