mizvekov added a comment.

@rsmith:

Actually, there is still something not quite right with this patch that I have 
to finish investigating.

The B5 <https://reviews.llvm.org/B5> test should not have gotten copy elision. 
I thought at first that we had gotten the function return type deduced during 
VarDecl instantiation, but we actually have a dependant type instead:

dump of FT:

  FunctionProtoType 0x18597a3ed10 '<dependent type> (void)' dependent cdecl
  `-BuiltinType 0x185960144e0 '<dependent type>' dependent

I think I assumed wrong that these blocks would behave similarly to lambdas 
with `auto` return.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99696

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

Reply via email to