Re: [PATCH] c: Fix ICE with cast to VLA [93576]

2020-02-13 Thread Jeff Law
On Thu, 2020-02-13 at 18:57 +0100, Jakub Jelinek wrote: > Hi! > > The following testcase ICEs, because the PR84305 changes try to evaluate > the size earlier. If size has side-effects, that is desirable, and the > side-effects will actually be wrapped in a SAVE_EXPR. The problem on this > testca

[PATCH] c: Fix ICE with cast to VLA [93576]

2020-02-13 Thread Jakub Jelinek
Hi! The following testcase ICEs, because the PR84305 changes try to evaluate the size earlier. If size has side-effects, that is desirable, and the side-effects will actually be wrapped in a SAVE_EXPR. The problem on this testcase is that there are no side-effects, and c_fully_fold doesn't fold