On Fri, 2014-05-23 at 17:45 +0400, Konstantin Serebryany wrote:
> On Fri, May 23, 2014 at 5:41 PM, Marek Polacek <pola...@redhat.com> wrote:
> > On Mon, May 12, 2014 at 03:20:37PM +0400, Konstantin Serebryany wrote:
> >> 5 months' worth of changes may break any platform we are not testing
> >> ourselves
> >> (that includes Ubuntu 12.04, 13.10, 14.04, Mac 10.9, Windows 7, Android
> >> ARM),
> >> please help us test this patch on your favorite platform.
> >
> > On powerpc64 I hit
> > /home/polacek/gcc/libsanitizer/asan/asan_linux.cc:209:3: error: #error
> > "Unsupported arch"
> > # error "Unsupported arch"
> >
> > because the merge (aka clang's r196802) removed ppc64 hunk of code:
> >
> > -# elif defined(__powerpc__) || defined(__powerpc64__)
> > - ucontext_t *ucontext = (ucontext_t*)context;
> > - *pc = ucontext->uc_mcontext.regs->nip;
> > - *sp = ucontext->uc_mcontext.regs->gpr[PT_R1];
> > - // The powerpc{,64}-linux ABIs do not specify r31 as the frame
> > - // pointer, but GCC always uses r31 when we need a frame pointer.
> > - *bp = ucontext->uc_mcontext.regs->gpr[PT_R31];
> > -# elif defined(__sparc__)
>
> Someone will have to send this patch via llvm-commits :(
> (I've pinged Peter Bergner once with no luck).
Sorry, I'm not purposely ignoring you. Just had a lot of other higher
priority items on my TODO list when you asked before. I'll try and
reapply the code that was removed and resubmit to the correct
mailing list.
Sorry for being so tardy and non-responsive.
Peter
