[PATCH] D153308: Analysis: Fix assertion when load alignment exceeds address space size

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 20 07:19:24 PDT 2023


gchatelet added a comment.

Actually, it should be `return OffsetTrailingZeroes >= MinimumTrailingZeroes || Offset.isZero();` because zero is always aligned to whatever power of two.
It probably makes sense to add a helper function in `Alignment.h`. I'll take care of it.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D153308/new/

https://reviews.llvm.org/D153308



More information about the llvm-commits mailing list