[PATCH] D17762: Fix an assertion failure in setPointOfInstantiation.
Haojian Wu via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 7 01:59:57 PST 2016
hokein added a comment.
In http://reviews.llvm.org/D17762#367182, @bkramer wrote:
> This doesn't look right to me. Aborting tree transform on a bad source location seems just wrong. Where is the invalid source location coming from? Also the backtrace seems related to typo correction, maybe the bug is there?
I have post the whole stack trace in PR25668 <https://llvm.org/bugs/show_bug.cgi?id=25668>.
The invalid source location comes from `TypoExpr` which is returned by `ArraySubscriptExpr->getLHS()`, this is also why there is a FIXME in code line <http://clang.llvm.org/doxygen/TreeTransform_8h_source.html#l08352>. After reading the code more throughly, I find that the invalid source location is expected when constructing a `TypoExpr`.
I updated the fixing patch now. We probably don't call `RequireCompleteType` with an invalid source location, which is the same as trunk at 84793.
http://reviews.llvm.org/D17762
More information about the cfe-commits
mailing list