From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:52804) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UQAIm-00084w-0q for qemu-devel@nongnu.org; Thu, 11 Apr 2013 01:44:05 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UQAIk-0000Fm-1H for qemu-devel@nongnu.org; Thu, 11 Apr 2013 01:44:03 -0400 Received: from [222.73.24.84] (port=18786 helo=song.cn.fujitsu.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UQAIj-0000Af-KI for qemu-devel@nongnu.org; Thu, 11 Apr 2013 01:44:01 -0400 Received: from fnstmail02.fnst.cn.fujitsu.com (tang.cn.fujitsu.com [127.0.0.1]) by tang.cn.fujitsu.com (8.14.3/8.13.1) with ESMTP id r3B5hmCt017217 for ; Thu, 11 Apr 2013 13:43:53 +0800 From: li guang In-Reply-To: <1365651662.9553.161.camel@liguang.fnst.cn.fujitsu.com> References: <1365651662.9553.161.camel@liguang.fnst.cn.fujitsu.com> Date: Thu, 11 Apr 2013 13:42:32 +0800 Message-ID: <1365658952.9553.169.camel@liguang.fnst.cn.fujitsu.com> Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="UTF-8" Subject: Re: [Qemu-devel] [PATCH 00/16] minially change gen_intermediate_code_internal function List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel Oh, sorry, minially -> minimally=20 =E5=9C=A8 2013-04-11=E5=9B=9B=E7=9A=84 11:41 +0800=EF=BC=8Cli guang=E5=86= =99=E9=81=93=EF=BC=9A > here are instructions in the finial qemu-system-x86 > before and after my chanage, you can easily find a=20 > redundant 'addl' was removed >=20 > this patch-set is v2 of previous 'translate: cleanup > gen_intermediate_code_internal patch' >=20 > v2: Keep variable 'num_insns' initailization > spotted by =E9=99=B3=E9=9F=8B=E4=BB=BB (Wei-Ren Chen) >=20 >=20 > before change: >=20 > 38c415: 3b 45 d4 cmp -0x2c(%ebp),%eax > 38c418: 7d 23 jge 38c43d > > ------------------------------------------------------------------ > 38c41a: 83 45 d0 01 addl $0x1,-0x30(%ebp)=20 > ------------------------------------------------------------------ > 38c41e: eb 15 jmp 38c435 > > 38c420: 8b 45 d0 mov -0x30(%ebp),%eax > 38c423: 8b 93 48 15 00 00 mov 0x1548(%ebx),%edx > 38c429: c6 84 02 a8 0e 01 00 movb $0x0,0x10ea8(%edx,%eax,1) > 38c430: 00=20 > 38c431: 83 45 d0 01 addl $0x1,-0x30(%ebp) > 38c435: 8b 45 d0 mov -0x30(%ebp),%eax > 38c438: 3b 45 d4 cmp -0x2c(%ebp),%eax > 38c43b: 7c e3 jl 38c420 > >=20 >=20 > after change: >=20 > 38c415: 3b 45 d4 cmp -0x2c(%ebp),%eax > 38c418: 7d 1f jge 38c439 > > 38c41a: eb 11 jmp 38c42d > > 38c41c: 8b 45 d0 mov -0x30(%ebp),%eax > 38c41f: 8b 93 48 15 00 00 mov 0x1548(%ebx),%edx > 38c425: c6 84 02 a8 0e 01 00 movb $0x0,0x10ea8(%edx,%eax,1) > 38c42c: 00=20 > 38c42d: 83 45 d0 01 addl $0x1,-0x30(%ebp) > 38c431: 8b 45 d0 mov -0x30(%ebp),%eax > 38c434: 3b 45 d4 cmp -0x2c(%ebp),%eax > 38c437: 7c e3 jl 38c41c > >=20 >=20 >=20