[PATCH] D47327: Check that memory allocation succeeds before use.
Tom Rix via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 24 13:13:46 PDT 2018
trixirt added a comment.
As llvm is mostly an api, i assumed that it would be robust and this was a one-off.
Anything big should be done automagically, so i am going to look at sic-ing clang-tidy and/or the static analyzer at this problem.
The lack of exception handling i can understand for performance reasons but it does put normal use of the std library in a bind if like this case the default uses exceptions.
It would be nice if there was project wide design on how to handle exceptions/errors.
If there is please point me at it.
Repository:
rL LLVM
https://reviews.llvm.org/D47327
More information about the llvm-commits
mailing list