CodeGen: block capture shouldn't ICE

CodeGen: block capture shouldn't ICE

When a lambda capture captures a __block in the same statement, the compiler asserts out because isCapturedBy assumes that an Expr can only be a BlockExpr, StmtExpr, or if it's a Stmt then all the statement's children are expressions. That's wrong, we need to visit all sub-statements even if they're not expressions to see if they also capture.

Fix this issue by pulling out the isCapturedBy logic to use RecursiveASTVisitor.

<rdar://problem/39926584>

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@332801 91177308-0d34-0410-b5e6-96231b3b80d8