2017-03-15 20:09 GMT+03:00 Jeff Law <l...@redhat.com>: > On 03/15/2017 04:15 AM, Martin Liška wrote: >> >> On 03/14/2017 11:27 PM, Ilya Enkovich wrote: >>> >>> 2017-03-10 16:15 GMT+03:00 Martin Liška <mli...@suse.cz>: >>>> >>>> Hello. >>>> >>>> Currently, __builtin_ia32_bndret is used for all functions that have >>>> non-void >>>> return type. I think the right fix is to return bounds just for a >>>> bounded type. >>>> >>>> Patch can bootstrap on x86_64-linux-gnu and survives regression tests. >>>> >>>> Ready to be installed? >>>> Martin >>> >>> >>> Hi, >>> >>> The fix makes sense and allows you to pass the test but it seems like we >>> still have the issue in inlining which can result in bndret call with >>> wrong arg. >> >> >> Hi. >> >> The test I added does probably what you mean: a return value of integer >> is casted >> to a pointer type. Or? >> >>> Do you avoid all such cases by this fix? Can we still have similar >>> problem >>> if cast function with integer return type to a function with pointer >>> return type >>> and the inline the call (not sure if we can inline such calls)? >>> >>> I think this fix still should go to trunk anyway. >> >> >> Good, may I consider this as patch approval? > > Yes. Ilya knows this stuff better than anyone, even if he's not listed as > an official maintainer.
I'm still in the list :) Ilya > > jeff >
