[PATCH] D154007: Reland "Try to implement lambdas with inalloca parameters by forwarding without use of inallocas."

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 28 15:30:41 PDT 2023


efriedma added a comment.

I'm not confident that isUsed() works the way you want it to in this context.  In particular, if the code in question runs before we've translated the whole translation unit, the isUsed() bit could change.  If you want that's more obviously safe, you could just check if there are any captures.  (I'm assuming the point of this is just to reduce the number of lambdas that go through this codepath?)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154007



More information about the cfe-commits mailing list