On Thu, Mar 23, 2017 at 08:00:11PM +0300, Alexander Monakov wrote: > On Thu, 23 Mar 2017, Jakub Jelinek wrote: > > And then clear it. That doesn't look like the right thing. > > > > So either you need some bool variable whether you've actually allocated > > the vector in the current expand_call_inline and use that instead of > > if (id->dst_simt_vars), or maybe you should clear id->dst_simt_vars > > otherwise and save/restore it around unconditionally. > > Yes, thanks for catching this. I went for the latter approach in the > following > patch.
Ok for trunk, thanks. For the nvptx bits, I think you need to ask Bernd to review it. Jakub