[PATCH] D123319: Change how we handle auto return types for lambda operator() to be consistent with gcc

Shafik Yaghmour via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 11 10:21:51 PDT 2022


shafik added a comment.

> Any update/further details here?

David, apologies for not getting back to you sooner. The context is D105564 <https://reviews.llvm.org/D105564> which I started working on again recently. I was having difficulties finding a solution that also worked for local lambdas. I discovered eventually that what I had worked with debug-info that gcc generated and realized that gcc was not emitting `auto` for lambdas and decided to match gcc's behavior here since we often do that. I think an alternative approach would have been to emit `DW_AT_linkage_name` for local lambdas but when I dug into that it looked like we were dropping the linkage name several step before where would emit `DW_AT_linkage_name` and it was not clear to me how easy a fix that was.

I am happy to consider other approaches as well to solving lookup for local lambdas for D105564 <https://reviews.llvm.org/D105564>. Let me know what you think.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123319



More information about the cfe-commits mailing list