Date: Tue, 22 Sep 2015 08:23:07 -0600 From: Ian Lepore <ian@freebsd.org> To: Dimitry Andric <dim@FreeBSD.org> Cc: Slawa Olhovchenkov <slw@zxy.spb.ru>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r288099 - head/contrib/binutils/opcodes Message-ID: <1442931787.1224.205.camel@freebsd.org> In-Reply-To: <4CD0DC75-0235-4890-827C-4FF41BBADD8A@FreeBSD.org> References: <201509220935.t8M9ZZfD051027@repo.freebsd.org> <20150922094301.GW3158@zxy.spb.ru> <4CD0DC75-0235-4890-827C-4FF41BBADD8A@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 2015-09-22 at 11:53 +0200, Dimitry Andric wrote: > On 22 Sep 2015, at 11:43, Slawa Olhovchenkov <slw@zxy.spb.ru> wrote: > > > > On Tue, Sep 22, 2015 at 09:35:35AM +0000, Dimitry Andric wrote: > ... > >> - imm |= (-1 << 7); > >> + imm |= -(1 << 7); > > > > May be (~0 << 7) is more simple to understund? > > This will give the same warning. ~0 will implicitly convert to -1. > > (It would be better to convert all these masks to unsigned, where such > shifting is always defined, but that gives a lot more churn.) > > -Dimitry > How is (-1U << 7) more churn? -- Ian
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1442931787.1224.205.camel>