Fixing temporary variables that are going out of scope in Release build execution

Open khuck opened this issue 5 years ago • 2 comments

see #1187

Jun 02 '20 15:06 khuck

Are you sure those values returned from .tensor() went out of scope too early? The code here looks ok without your changes (but those shouldn't hurt in any way either).

Jun 02 '20 17:06 hkaiser

Are you sure those values returned from .tensor() went out of scope too early? The code here looks ok without your changes (but those shouldn't hurt in any way either).

Honestly, because it's a Release build I couldn't get any info debugging. So I just made some assumptions. I am being a bit more disciplined with it now though - the debug build also fails with the assertion error, so I am replacing temporaries one at a time until I find the sweet spot between correct execution (no assertion error) and not crashing.

Jun 02 '20 17:06 khuck