On 10/17/2014 05:49 PM, Jakub Jelinek wrote: > Plus if you add misalign tests...
Sure, can do this on Monday. > - bool use_calls = ASAN_INSTRUMENTATION_WITH_CALL_THRESHOLD < INT_MAX
- && asan_num_accesses >= ASAN_INSTRUMENTATION_WITH_CALL_THRESHOLD; + bool use_calls + = ASAN_INSTRUMENTATION_WITH_CALL_THRESHOLD < INT_MAX + && (flag_sanitize & SANITIZE_KERNEL_ADDRESS) + && asan_num_accesses >= ASAN_INSTRUMENTATION_WITH_CALL_THRESHOLD;
I agree that original code didn't quite match GNU conventions but can we avoid reformatting it to make future backports easier? So e.g.
bool use_calls = ASAN_INSTRUMENTATION_WITH_CALL_THRESHOLD < INT_MAX + && (flag_sanitize & SANITIZE_KERNEL_ADDRESS) && asan_num_accesses >= ASAN_INSTRUMENTATION_WITH_CALL_THRESHOLD; -Y