[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