Previously, a warning that C++20 deprecated implicit capture of 'this' for lambda captures specified with a capture default of '=' was only issued when '-Wdeprecated' or '-Wdeprecated-this-capture' was specified. This change enables the warning by default (it is still only issued when compiling for C++20 or later). This is consistent with gcc which warns by default (MSVC requires '/Wall').
You'll need a rebase to get the pre-commit CI to work. Else, looks fine to me. Make sure @royjacobson knows of the change for this (https://reviews.llvm.org/D142578).
You'll need a rebase to get the pre-commit CI to work.
Yes, thank you. I'll wait for https://reviews.llvm.org/D142578 to land and then rebase.