[PATCH] D66924: [NewGVN] Add phi-of-ops instr as user of FoundVal.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 28 21:49:34 PDT 2019


fhahn created this revision.
fhahn added reviewers: efriedma, davide.
Herald added subscribers: hiraditya, Prazek.
Herald added a project: LLVM.

If any of the values we chose as translated PHI operands
change, we have to re-evaluate the phi-of-ops for
I. We already record additional dependencies for
the phi translated operands, but we miss the link
to the translated operand. Those can change without
without their operands changing, e.g. if the leader
of their congruence class changes.

Even with this patch, there are still some dependencies
missing. I will continue tackling the outstanding
reporeted crashes in this area.

Fixes PR36501, PR42422, PR42557.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D66924

Files:
  llvm/lib/Transforms/Scalar/NewGVN.cpp
  llvm/test/Transforms/NewGVN/pr36501-changed-after-3.ll
  llvm/test/Transforms/NewGVN/pr42422-changed-after-2.ll
  llvm/test/Transforms/NewGVN/pr42557-changed-after-1.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D66924.217770.patch
Type: text/x-patch
Size: 17847 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190829/eef190a1/attachment-0001.bin>


More information about the llvm-commits mailing list