[PATCH] D42366: [CodeGen] Fix generation of TBAA tags for may-alias accesses

Ivan Kosarev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 21 02:19:35 PST 2018


kosarev added a comment.

I think zero would serve better as the unknown-size value. People who are not aware of TBAA internals would guess that since zero-sized accesses make no sense, they are likely to have some special meaning. Similarly, for code that is supposed to process the size fields of access descriptors zero would be an obvious "illegal size value". In contrast, UINT64_MAX is just a very large number that doesn't hint anything on its special purpose.

Either way, we should reflect the convention in the documentation, https://reviews.llvm.org/D40975.


Repository:
  rC Clang

https://reviews.llvm.org/D42366





More information about the cfe-commits mailing list