Re: try_finally_expr wrong source location info

2017-07-12 Thread Vyacheslav Barinov
x27;t break GCC testsuite. So I'll send it here, maybe someone else needs the fix. Best Regards, Vyacheslav Barinov Eric Botcazou writes: >> But then, after lowering on the eh pass (gcc/tree-eh.c:1161), the "finally" >> location is set up to the `gimple_location (tf->tr

Re: try_finally_expr wrong source location info

2017-06-13 Thread Vyacheslav Barinov
ly to GCC online docs it shouldn't: > the C++ front end uses pseudo-scopes to handle cleanups for objects with > destructors, these don’t translate into the GIMPLE form The best thing I could achieve is to set `finally' source location equal to the previous statement. Now my report shows to the last line in function, which is much better than blaming constructor, but still not the result I wanted to get; this might be confusing when last line is some kind of resource freeing function call. Best Regards, Vyacheslav Barinov

try_finally_expr wrong source location info

2017-06-09 Thread Vyacheslav Barinov
try_finally_onedest', but unsuccessfully. And also, as far as I understand, this issue should be fixed in all the lowering functions, not only this one. Where should I dig in order to fix the issue? Best Regards, Vyacheslav Barinov