Bug 342008

Summary:	valgrind.h needs type cast for _zzq_default to compile with clang/llvm in 64-bit mode
Product:	[Developer tools] valgrind	Reporter:	chh
Component:	general	Assignee:	Julian Seward <jseward>
Status:	RESOLVED FIXED
Severity:	normal	CC:	dimitry
Priority:	NOR
Version First Reported In:	3.10.0
Target Milestone:	---
Platform:	Android
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description chh 2014-12-18 18:58:43 UTC

valgrind.h got a compiler warning/error from clang/llvm integrated assembler.
A potential fix for Android is in:
  https://android-review.googlesource.com/#/c/119291/1

which added a missing type cast for
      : "r" (_zzq_default), "r" (&_zzq_args[0]) 
to 
     : "r" ((unsigned long int)_zzq_default), "r" (&_zzq_args[0]) 

For the latest valgrind.h, type o _zzq_default is changed to unsigned long long int.

Could someone confirm this change and make it in valgrind?
Thanks.

Comment 1 Julian Seward 2015-09-06 13:11:00 UTC

Committed, r15633.  Thanks for the patch.