[PATCH] D82365: [Power10] Implement Vector Insert Builtins in LLVM/Clang

Mon Jul 6 12:54:02 PDT 2020

lei added inline comments.

================
Comment at: clang/test/CodeGen/builtins-ppc-p10vector.c:12
+// RUN:   -target-cpu pwr10 -triple powerpc64le-unknown-unknown -emit-llvm %s \
+// RUN:   -o - | FileCheck %s -check-prefix=CHECK-LE
+
----------------
biplmish wrote:
> lei wrote:
> > I just noticed this.  There is no need to add this RUN line since it's the same as the one on line 2.  Please post a patch to remove this and update tests to use the default `CHECK`.
> Sure. However there are also tests in Line 125,133 etc which would need modification.
> 
> Can we also do "RUN:   -o - | FileCheck %s -check-prefixes=CHECK,CHECK-LE" in the test1 and remove the test3 so that the tests work in the current format.
Please make all the necessary modification all affected testcases to use `CHECK` instead of `CHECK-LE`.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82365