Re: [PATCH] D30312: Fix unix.Malloc analysis crasher when allocating dynamic arrays w/unbound statements (fix PR32050)

2017-02-23 Thread Kevin Marshall via cfe-commits
Apparently not - I'm at 289944 locally. Sigh. Oh well, it was an interesting investigation. On Thu, Feb 23, 2017 at 3:48 PM, Nico Weber wrote: > This looks pretty similar to https://reviews.llvm.org/D27849 – are you > synced to trunk? > > On Thu, Feb 23, 2017 at 5:42 PM,

Re: [PATCH] D30312: Fix unix.Malloc analysis crasher when allocating dynamic arrays w/unbound statements (fix PR32050)

2017-02-23 Thread Nico Weber via cfe-commits
This looks pretty similar to https://reviews.llvm.org/D27849 – are you synced to trunk? On Thu, Feb 23, 2017 at 5:42 PM, Kevin Marshall via Phabricator via cfe-commits wrote: > kmarshall created this revision. > > The extent calculation function had a bug which

[PATCH] D30312: Fix unix.Malloc analysis crasher when allocating dynamic arrays w/unbound statements (fix PR32050)

2017-02-23 Thread Kevin Marshall via Phabricator via cfe-commits
kmarshall created this revision. The extent calculation function had a bug which caused it to ignore if the size value was defined prior to casting it. As a result, size expressions with free variables would trigger assertion failures during the cast operation. This patch adds that missing