[Type checker] Don't replace function builder closures with their bodies.
When diagnosing failures in a function builder closure, we were
unnecessarily replacing the closure expression with its body,
destroying the AST and resulting in assertions due to DeclContext
mismatches. Fixes SR-11350 / rdar://problem/54590425.
(cherry picked from commit 187647de62fc6132c4e4c6f891edeba3133834e2)