================
@@ -83,7 +83,13 @@ __llvm_profile_iterate_data(const __llvm_profile_data *Data) 
{
 /* This method is only used in value profiler mock testing.  */
 COMPILER_RT_VISIBILITY void *
 __llvm_get_function_addr(const __llvm_profile_data *Data) {
-  return Data->FunctionPointer;
+  void *FP = Data->FunctionPointer;
+#if __has_feature(ptrauth_calls)
+  // This is only used for tests where we compare against what happens to be
+  // signed pointers.
+  FP = ptrauth_sign_unauthenticated(FP, VALID_CODE_KEY, 0);
----------------
kovdan01 wrote:

> @kovdan01 I mean "do you have step by step instructions for a muppet like me 
> that would make it possible for me to build/test locally" :D

@ojhunt Ah, thanks for clarifying your question :) Unfortunately, right now our 
build scripts, instructions, tests, ... are not available publicly. We'll 
definitely publish them, but right now I guess that we would be happy to test 
your changes on our side manually. This would, of course, slow down the work on 
the PR, but at this point it looks like that many of the comments we left 
actually are relevant for all targets, not only for linux. So, at least for 
such comments you'll be able to test things using your system w/o our 
linux-specific stuff.

https://github.com/llvm/llvm-project/pull/143230
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to