[PATCH] D69733: [PowerPC] Remove allow-deprecated-dag-overlap and fix broken tests

Jinsong Ji via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 1 13:09:24 PDT 2019


jsji created this revision.
jsji added reviewers: PowerPC, hfinkel, nemanjai, steven.zhang.
Herald added subscribers: llvm-commits, shchenz.
Herald added a project: LLVM.

This is found during review of https://reviews.llvm.org/D67088.

CHECK-DAG is non-overlapping after https://reviews.llvm.org/D47106.
-allow-deprecated-dag-overlap was introduced to temporary accept old
behavior.

But it actually hide some broken tests, eg: `test/CodeGen/PowerPC/swaps-le-1.ll`
The codegen has changed, but the CHECK-DAG still PASS due to allowing `overlap`.

This patch remove the deprecated options, and fix the broken tests.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D69733

Files:
  llvm/test/CodeGen/PowerPC/build-vector-tests.ll
  llvm/test/CodeGen/PowerPC/machine-combiner.ll
  llvm/test/CodeGen/PowerPC/memcpy-vec.ll
  llvm/test/CodeGen/PowerPC/swaps-le-1.ll
  llvm/test/CodeGen/PowerPC/uint-to-fp-v4i32.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69733.227517.patch
Type: text/x-patch
Size: 7764 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191101/5172ead2/attachment.bin>


More information about the llvm-commits mailing list