[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