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

2020-04-05 Thread Tyker via Phabricator via cfe-commits
Tyker added a comment. In 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

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

2020-04-04 Thread Johannes Doerfert via Phabricator via cfe-commits
jdoerfert added a comment. @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.

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

2020-04-04 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri added a comment. What's the status here? 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

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

2019-12-19 Thread pre-merge checks [bot] via Phabricator via cfe-commits
merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 60947 tests passed, 20 failed and 726 were skipped. failed: Clang.CodeGen/align_value.cpp failed: Clang.CodeGen/builtin-assume-aligned.c failed: Clang.CodeGen/builtin-movdir.c failed:

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

2019-12-19 Thread Johannes Doerfert via Phabricator via cfe-commits
jdoerfert updated this revision to Diff 234817. jdoerfert added a comment. Rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71739/new/ https://reviews.llvm.org/D71739 Files: clang/lib/CodeGen/CodeGenFunction.cpp

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

2019-12-19 Thread pre-merge checks [bot] via Phabricator via cfe-commits
merge_guards_bot added a comment. {icon question-circle color=gray} Unit tests: unknown. {icon question-circle color=gray} clang-tidy: unknown. {icon question-circle color=gray} clang-format: unknown. Build artifacts :

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

2019-12-19 Thread Johannes Doerfert via Phabricator via cfe-commits
jdoerfert created this revision. jdoerfert added reviewers: hfinkel, xbolva00, lebedev.ri, nikic, Tyker, rjmccall, spatel. Herald added subscribers: bollu, hiraditya. Herald added projects: clang, LLVM. NOTE: This is a prototype not a finished patch! NOTE: There is a mailing list discussion on