royjacobson added a comment.

Thanks for picking this up! :)

The (non-wording) paper makes a pretty convincing case to just apply this 
retroactively to any C++11 code 
(https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2022/p2644r1.pdf). I think 
we should apply this retroactively, maybe add a pedantic warning when we do a 
lifetime extension on code before C++23.

Also, +1 to more tests. Specifically constexpr and dependent contexts would be 
good to see.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139586

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

Reply via email to