[clang] [Clang][Sema] Fix an ICE where structured binding packs within a lambda should be diagnosed immediately (PR #214716)

[via cfe-commits]

================
@@ -369,17 +358,17 @@ features cannot lower the translation-unit ABI level;
 
 ### Bug Fixes in This Version
 
-- Fixed incorrect handling of C++ import preprocessing token when a digraph character after import. (#GH190693)
 - Fixed an assertion failure when passing a wide string literal to `__builtin_nan`. (#GH212108)
 - Fixed a constraint comparison bug in partial ordering. (#GH182671)
 - Fixed a rejected-valid case that used an explicit object parameter in an out-of-line definition of a nested class member. (#GH136472)
 - Fixed an assertion on omp taskloop transparent (#GH197162)
 - Fixed a bug where `__func__`, `__PRETTY_FUNCTION__` and `__FUNCTION__` were not resolving to the proper function when inside a lambda return type (#GH211811)
 - Fixed USR generation for declarations whose signature mentions a class-type
   non-type template parameter. (#GH212351)
-- Fixed an assertion crash when instantiating a nested requirement with an invalid constraint. (#GH213575)
 - Clang now defines the GCC-compatible predefined macro `__SIG_ATOMIC_TYPE__`. (#GH213895)
-- Fixed a bug where a stray closing curley brace in an OpenMP/OpenACC pragma could cause pragma parsing issues when inside of a member function. (#GH214195)
+- Fixed an ICE where invalid structured binding packs within a lambda were
----------------
babadany2999 wrote:

It is more accurate that the fix was adding structured binding packs within a lambda to the local pack.... should this change? Idk. The fact that the diagnostics were delayed is merely a symptom and was not the actual issue... but user-facing release notes should not care about that i think ?
Waiting for reviewers to weigh in on this!

https://github.com/llvm/llvm-project/pull/214716