[PATCH] D76550: [Attributor] Improve the alignment of the loads

omar ahmed via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 21 13:52:03 PDT 2020


omarahmed added a comment.

In D76550#1935236 <https://reviews.llvm.org/D76550#1935236>, @jdoerfert wrote:

> You should query the base alignment AA in the initialize once to make sure it exists and is part of the fixpoint iteration. For now that is trivially the case but I can see that this might not be this way in the future.


Sry but you mean we should query the base in createInitialization right or you mean another initialization ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76550





More information about the llvm-commits mailing list