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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
