[all-commits] [llvm/llvm-project] 228dd9: [PowerPC] Remove allow-deprecated-dag-overlap and ...

jsji via All-commits all-commits at lists.llvm.org
Tue Nov 12 07:26:09 PST 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 228dd96c6fddad6d070c29a6830c7bd161982e3a
      https://github.com/llvm/llvm-project/commit/228dd96c6fddad6d070c29a6830c7bd161982e3a
  Author: Jinsong Ji <jji at us.ibm.com>
  Date:   2019-11-12 (Tue, 12 Nov 2019)

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

  Log Message:
  -----------
  [PowerPC] Remove allow-deprecated-dag-overlap and fix broken tests

Summary:
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.

Reviewers: #powerpc, hfinkel, nemanjai, steven.zhang, shchenz

Reviewed By: shchenz

Subscribers: shchenz, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D69733




More information about the All-commits mailing list