[PATCH] D95094: [ShrinkWrap] Enable Shrinkwrapping on 32 and 64 bit AIX, and 64 bit ELF

Sean Fertile via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 10 07:29:17 PST 2021


sfertile added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/ppc-shrink-wrapping.ll:3
+; RUN: llc -mtriple=powerpc64le-unknown-linux-gnu %s -o - -enable-shrink-wrap=false -verify-machineinstrs |  FileCheck %s --check-prefix=CHECK-COM --check-prefix=DISABLE-COM --check-prefix=CHECK-LINUX --check-prefix=DISABLE-LINUX
+; RUN: llc -mtriple=powerpc-ibm-aix-xcoff -mattr=-altivec -mcpu=pwr4 %s -o - -verify-machineinstrs | FileCheck %s --check-prefix=CHECK-COM --check-prefix=ENABLE-COM --check-prefix=CHECK-AIX --check-prefix=ENABLE-AIX --check-prefix=CHECK-32AIX --check-prefix=ENABLE-32AIX 
+; RUN: llc -mtriple=powerpc-ibm-aix-xcoff %s -o - -enable-shrink-wrap=false -verify-machineinstrs |  FileCheck %s --check-prefix=CHECK-COM --check-prefix=DISABLE-COM --check-prefix=CHECK-AIX --check-prefix=DISABLE-AIX --check-prefix=CHECK-32AIX --check-prefix=DISABLE-32AIX
----------------
A lot of lines in this patch add whitespace error, I get the following message when applying this patch with git:
```
warning: squelched 28 whitespace errors
warning: 33 lines add whitespace errors.
```

This line adds one, but there are too many to call out each individually on phab. Can you open the files that were modified in this patch in vim and highlight trailing whitespace and remove it on any line that was modified?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95094/new/

https://reviews.llvm.org/D95094



More information about the llvm-commits mailing list