[PATCH] D69900: Align load/store for llvm.loop.aligned metadata

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

In D69900#1736112 <https://reviews.llvm.org/D69900#1736112>, @hfinkel wrote:

> Do we need this at all? Can't we just emit `@llvm.assume` calls before the loop if we know the alignment? I've done this in the past and it's worked well (by using __builtin_assume_aligned  from Clang).


I'm in favor of doing this (`llvm.assume`) in the frontedn so other passes can use the information as well without requiring them to understand yet another encoding.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69900