[PATCH 2/3] Simplify and inline get_uleb128 and get_sleb128

Mark Wielaard mjw at redhat.com
Tue Apr 22 22:04:31 UTC 2014


On Tue, 2014-04-22 at 08:58 -0700, Richard Henderson wrote:
> On 04/22/2014 08:52 AM, Josh Stone wrote:
> > So in total:
> > 
> >   struct { signed int i:7; } __s = { .i = __b };
> >   (var) |= (typeof (var)) __s.i * ((typeof (var)) 1 << ((nth) * 7));
> > 
> > Better?
> 
> Yes, that's what I had in mind.

Excellent. I changed the patch as attached. No undefined behavior
detected, all tests PASS.

Thanks,

Mark

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-libdw-get_sleb128_step-Remove-undefined-behavior.patch
Type: text/x-patch
Size: 1844 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/elfutils-devel/attachments/20140423/ba1f1fd7/attachment-0001.bin>


More information about the elfutils-devel mailing list