[PATCH] D35851: [Dominators] Include infinite loops in PostDominatorTree

[Jakub Kuderski via Phabricator via llvm-commits]

kuhar updated this revision to Diff 111088.
kuhar added subscribers: uabelho, kparzysz.
kuhar added a comment.
Herald added subscribers: javed.absar, nhaehnle.

When running tests for all targets, I discovered that 3 codegen tests got affected by this patch.

I think it would be best if someone took a look at the changes -- I'm especially concerned about register spilling in `test/CodeGen/ARM/struct-byval-frame-index.ll`. I can see that the test was updates a few days ago by https://reviews.llvm.org/D34099.
@uabelho, @kparzysz, could you take a look?


https://reviews.llvm.org/D35851

Files:
  include/llvm/Support/GenericDomTree.h
  include/llvm/Support/GenericDomTreeConstruction.h
  lib/Transforms/Scalar/ADCE.cpp
  test/Analysis/PostDominators/infinite-loop.ll
  test/Analysis/PostDominators/infinite-loop2.ll
  test/Analysis/PostDominators/infinite-loop3.ll
  test/Analysis/PostDominators/pr24415.ll
  test/Analysis/PostDominators/pr6047_a.ll
  test/Analysis/PostDominators/pr6047_b.ll
  test/Analysis/PostDominators/pr6047_c.ll
  test/Analysis/PostDominators/pr6047_d.ll
  test/Analysis/RegionInfo/infinite_loop.ll
  test/Analysis/RegionInfo/infinite_loop_2.ll
  test/Analysis/RegionInfo/infinite_loop_3.ll
  test/Analysis/RegionInfo/infinite_loop_4.ll
  test/Analysis/RegionInfo/infinite_loop_5_a.ll
  test/Analysis/RegionInfo/infinite_loop_5_b.ll
  test/CodeGen/AMDGPU/branch-relaxation.ll
  test/CodeGen/ARM/struct-byval-frame-index.ll
  test/CodeGen/Thumb2/v8_IT_5.ll
  test/Transforms/StructurizeCFG/branch-on-argument.ll
  test/Transforms/StructurizeCFG/no-branch-to-entry.ll
  unittests/IR/DominatorTreeTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35851.111088.patch
Type: text/x-patch
Size: 51550 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170814/a56e0248/attachment.bin>