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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to