NoQ abandoned this revision.
NoQ added a comment.
Herald added subscribers: mikhail.ramalho, rnkovacs, szepet.
Herald added a reviewer: george.karpenkov.
Outdated by https://reviews.llvm.org/D44131.
https://reviews.llvm.org/D27202
___
cfe-commits
NoQ planned changes to this revision.
NoQ added a comment.
Yep, because there are a lot more places in which the value of the expression
suddenly changes, i think i'd want to either fix all of them, or (if i find
some of those really reasonable) make a callback for checkers to subscribe to
zaks.anna added a comment.
From what I recall, it is not clear that this patch is the step in the right
direction. At least, it need more investigation.
https://reviews.llvm.org/D27202
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
zaks.anna added a comment.
> Could you please remove the IteratorPastEndChecker file differences from this
> patch and make https://reviews.llvm.org/D25660
> dependent on this one?
Since this has not been reviewed yet while the IteratorPastEndChecker has been,
I do not think we should block
baloghadamsoftware added a comment.
Could you please remove the IteratorPastEndChecker file differences from this
patch and make https://reviews.llvm.org/D25660 dependent on this one?
https://reviews.llvm.org/D27202
___
cfe-commits mailing list
NoQ created this revision.
NoQ added reviewers: zaks.anna, dcoughlin, xazax.hun, a.sidorin.
NoQ added subscribers: cfe-commits, baloghadamsoftware.
Instead, return a `nonloc::LazyCompoundVal` of a temporary region for the call
expression, with a trivial store in which the temporary region is
