diff options
| author | Angel Garcia Gomez <angelgarcia@google.com> | 2015-10-05 08:40:13 +0000 |
|---|---|---|
| committer | Angel Garcia Gomez <angelgarcia@google.com> | 2015-10-05 08:40:13 +0000 |
| commit | c50b9fbb84832d5f697155a6f756873deec86e95 (patch) | |
| tree | b4863421c0be6585ead00b70752b0573e1c928cf /clang/lib/Basic/FileManager.cpp | |
| parent | 28ff7287874eaf4eb7c1c6667a6f9978ba0cc6bd (diff) | |
| download | llvm-c50b9fbb84832d5f697155a6f756873deec86e95.zip llvm-c50b9fbb84832d5f697155a6f756873deec86e95.tar.gz llvm-c50b9fbb84832d5f697155a6f756873deec86e95.tar.bz2 | |
Fix bug in modernize-use-nullptr.
Summary:
https://llvm.org/bugs/show_bug.cgi?id=24960
modernize-use-nullptr would hit an assertion in some cases involving macros and initializer lists, due to finding a node with more than one parent (the two forms of the initializer list).
However, this doesn't mean that the replacement is incorrect, so instead of just rejecting this case I tried to find a way to make it work. Looking at the semantic form of the InitListExpr made sense to me (looking at both forms results in false negatives) but I am not sure of the things that we can miss by skipping the syntactic form.
Reviewers: klimek
Subscribers: cfe-commits, alexfh
Differential Revision: http://reviews.llvm.org/D13246
llvm-svn: 249291
Diffstat (limited to 'clang/lib/Basic/FileManager.cpp')
0 files changed, 0 insertions, 0 deletions