[clang] [clang][AST] Added assert to prevent infinite recursion in computing layout (PR #154134)

[Michael Buch via cfe-commits]

Michael137 wrote:

Personally i think the assert makes sense here (if a user can't ever create an AST like that, which IIUC they can't). Although it is true that we try not to assert on user input as Jonas mentioned, this was a bug in our LLDB's AST creation. Malformed DWARF can already cause all sorts of Clang assertions to trigger anyway, so rejecting adding an assert into Clang just because LLDB may create an incorrect AST out of it seems counter-productive

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