[PATCH] D24238: StaticAnalyzer CastToStruct : No memory corruption when casting array to struct

Artem Dergachev via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 9 12:34:15 PDT 2016


NoQ added a subscriber: cfe-commits.
NoQ added a comment.

Adding cfe-commits as per developer policy.

Yeah, it doesn't probably cause the same kind of memory corruption, however i wouldn't call this code safe: it still violates the //strict aliasing rule//, unless the array is of `char`s. I think this warning is worth keeping for non-`char` arrays. Even with `-fno-strict-aliasing` of some sort, the programmer might run into endianness or alignment issues.


https://reviews.llvm.org/D24238





More information about the cfe-commits mailing list