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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
