[llvm] r296998 - [XRay] Allow logging the first argument of a function call.

Dean Michael Berris via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 7 15:14:38 PST 2017


Hi Bill,

There's a change being proposed in https://reviews.llvm.org/D30635 that
should XFAIL this test on powerpc. Let me land that and see if that fixes
it.

Sorry for the delay here.

On Wed, Mar 8, 2017 at 2:08 AM Bill Seurer <seurer at linux.vnet.ibm.com>
wrote:

> On 03/06/2017 09:00 PM, Martin Pelikán wrote:
> >>> Your change appears to cause an undefined symbol error when the test
> >>> cases are building on ppc64le.
> >
> >>>
> /home/buildbots/ppc64le-clang-test/clang-ppc64le/llvm/projects/compiler-rt/lib/xray/xray_interface.cc:200:
> >>> undefined reference to `__xray::patchFunctionEntry(bool, unsigned int,
> >>> XRaySledEntry const&, void (*)())'
> >
> > Yes, sorry.  This was fixed by D30634 and committed as soon as we
> > could when we realized the problem.  There's still the issue of
> > temporarily disabling tests for the unfinished implementations
> > (correctly) in D30635.  Apologies for the breakage.
>
> Is this for the ongoing xray test case failure like
>
> http://lab.llvm.org:8011/builders/clang-ppc64le-linux/builds/4145
>
> --
>
> -Bill Seurer
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170307/f8e332b3/attachment.html>


More information about the llvm-commits mailing list