On (19/10/30 15:22), glider@xxxxxxxxxx wrote:
> Clang may replace stackdepot_memcmp() with a call to instrumented bcmp(),
> which is exactly what we wanted to avoid creating stackdepot_memcmp().
> Add a compiler barrier() to prevent optimizations.
[..]
> @@ -163,6 +163,11 @@ int stackdepot_memcmp(const unsigned long *u1, const unsigned long *u2,
> unsigned int n)
> {
> for ( ; n-- ; u1++, u2++) {
> + /*
> + * Prevent Clang from replacing this function with a bcmp()
> + * call.
> + */
> + barrier();
> if (*u1 != *u2)
> return 1;
> }
Would 'volatile' do the trick?
-ss
