On Thu, Feb 26, 2015 at 01:55:09AM -0800, Chaoren Lin wrote:
> > How is a fix for 32 bit builds only a solution for this issue?
> 
> It's not. That's why I said "I think it's better to apply Azat's solution
> of using ArchSpec::IsCompatibleMatch() instead to prevent this issue from
> resurfacing on other platforms in the future."
> 
> Feel free to add yourself as a reviewer here: http://reviews.llvm.org/D7897

I'm fine with this patch, and indeed it must fix not only 32bit builds.

Cheers,
Azat.

> 
> On Thu, Feb 26, 2015 at 1:32 AM, René J.V. <rjvber...@gmail.com> wrote:
> 
> > On Thursday February 26 2015 06:27:35 bugzilla-dae...@llvm.org wrote:
> >
> > How is a fix for 32 bit builds only a solution for this issue?
> > I confirm Sylvestre's comment that the issue also affects 64 bit builds;
> > I'm running KUbuntu 14.04.2LTS and my own llvm/lldb PPA with
> > Debian/Experimental's 1:3.5.1-1~exp1 package.
> >
> > R.
> >
> > >http://llvm.org/bugs/show_bug.cgi?id=20400
> > >
> > >Chaoren Lin <chaor...@google.com> changed:
> > >
> > >           What    |Removed                     |Added
> >
> > >----------------------------------------------------------------------------
> > >             Status|NEW                         |RESOLVED
> > >         Resolution|---                         |FIXED
> > >
> > >--- Comment #13 from Chaoren Lin <chaor...@google.com> ---
> > >Configuring cmake with LLVM_DEFAULT_TARGET_TRIPLE=i386-linux-gnu seems to
> > fix
> > >this issue on 32 bit builds. The default of i686-pc-linux-gnu is
> > compatible but
> > >not necessarily identical to the inferior binaries, so I think it's
> > better to
> > >apply Azat's solution of using ArchSpec::IsCompatibleMatch() instead to
> > prevent
> > >this issue from resurfacing on other platforms in the future.

_______________________________________________
lldb-dev mailing list
lldb-dev@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev

Reply via email to