[PATCH] D71739: [WIP] Use operand bundles to encode alignment assumptions

Tyker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 5 11:44:43 PDT 2020


Tyker added a comment.

In D71739#1961508 <https://reviews.llvm.org/D71739#1961508>, @jdoerfert wrote:

> @lebedev.ri  We'd need to identify other uses of the alignment encoding in-tree so we can replace them as well. Also, this patch uses not only the alignment but also the offset in the operand bundle. We can either allow that or encode the offset via a gep in the IR. I guess the latter is easier to implement until we have more reasons to allow more complex operand bundles (which we will need to have eventually).


for now i think we will stay with the current "simple" alignment assumptions in operand bundles. but we can improve it later.

> @Tyker Do you want to take this?

i am fine with taking this. but there is a few thing to do before this.

i think that this patch depends on a few things:

- add an API to build assumes from provided knowledge.
- update users of alignment assumptions.

and a few things that i would like to do before:

- finish patches currently in review.
- improve generation of assume with operand bundles to minimize duplicates and extra instructions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71739





More information about the llvm-commits mailing list