r278264 - Reapply [Sema] Add sizeof diagnostics for bzero

[Bruno Cardoso Lopes via cfe-commits]

Hi Joerg,

> Given that bzero is pretty much non-standard at this point, I strongly
> dislike the approach of checking only the function name.

Thanks for the feedback. bzero it's widely used though, I think it's a
usability win to perform a diagnostic here, despite bzero being
non-standard.

Is there any additional reason why you dislike it or additional checks
you suggest? IIUC, Sema::CheckMemaccessArguments already checks if the
number of arguments is the same and if they are indeed what they're
supposed to be.

If deprecation is the main issue here, shouldn't clang emit warnings
to mention 'bzero' deprecation then?

-- 
Bruno Cardoso Lopes
http://www.brunocardoso.cc