[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