filcab added a comment. In https://reviews.llvm.org/D52615#1272725, @rjmccall wrote:
> In https://reviews.llvm.org/D52615#1266320, @filcab wrote: > > > In https://reviews.llvm.org/D52615#1254567, @rsmith wrote: > > > > > This seems like an unreasonably long flag name. Can you try to find a > > > shorter name for it? > > > > > > `-fsanitize-poison-extra-operator-new`? > > Not as explicit, but maybe ok if documented? > > > `-fsanitize-address-poison-array-cookie`? I strongly dislike this one because "poison array cookie", in general, is always enabled (it's actually triggered by a runtime flag). This flag is about poisoning it in more cases (cases where the standard doesn't completely disallow accesses to the cookie, so we have to have a flag and can't enable it all the time). Thank you, Filipe P.S: Some additional discussion is at https://reviews.llvm.org/D41664, from when this flag was first implemented. Repository: rC Clang https://reviews.llvm.org/D52615 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
