From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andi Kleen Subject: Re: [PATCH 17/17] perf annotate: Handle variables in 'sub', 'or' and many other instructions Date: Fri, 13 Apr 2018 09:20:43 -0700 Message-ID: <20180413162043.GD6694@tassilo.jf.intel.com> References: <20180413140111.25313-1-acme@kernel.org> <20180413140111.25313-18-acme@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20180413140111.25313-18-acme@kernel.org> Sender: linux-kernel-owner@vger.kernel.org To: Arnaldo Carvalho de Melo Cc: Ingo Molnar , Clark Williams , linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, Arnaldo Carvalho de Melo , Adrian Hunter , David Ahern , Jin Yao , Jiri Olsa , Martin =?utf-8?B?TGnFoWth?= , Namhyung Kim , Ravi Bangoria , Thomas Richter , Wang Nan List-Id: linux-perf-users.vger.kernel.org On Fri, Apr 13, 2018 at 11:01:11AM -0300, Arnaldo Carvalho de Melo wrote: > From: Arnaldo Carvalho de Melo > > Just like is done for 'mov' and others that can have as source or > targets variables resolved by objdump, to make them more compact: > > - orb $0x4,0x224d71(%rip) # 226ca4 <_rtld_global+0xca4> > + orb $0x4,_rtld_global+0xca4 That's not equivalent. It could be non rip relative too. You would need to keep at least the (%rip). -Andi