[PATCH] D81854: [InstCombine] Don't let an alignment assume prevent new/delete removals.
Hiroshi Yamauchi via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 17 12:24:14 PDT 2020
yamauchi added a comment.
In D81854#2096685 <https://reviews.llvm.org/D81854#2096685>, @jdoerfert wrote:
> This is the right fix for alignment: D71739 <https://reviews.llvm.org/D71739>
>
> For all `assumes` (and later other stuff, think lifetime markers or annotations) you can check if the `User::isDroppable`, and eventually drop via `Value::dropDroppableUses` if you cannot update them properly.
@lebedev.ri, @jdoerfert: I acknowledge D71739 <https://reviews.llvm.org/D71739>. What's the status of it? Are you open to having this patch as a temporary measure until that's checked in?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D81854/new/
https://reviews.llvm.org/D81854
More information about the llvm-commits
mailing list