[PATCH] D82467: [PowerPC][Power10] Implement Truncate and Store VSX Vector Builtins

Tue Jun 30 21:07:25 PDT 2020

amyk marked an inline comment as done.
amyk added inline comments.

================
Comment at: clang/test/CodeGen/builtins-ppc-p10vector.c:108
+                           signed char *__c) {
+  // CHECK-BE: store i8 %{{.+}}, i8* %{{.+}}, align 1
+  // CHECK-LE: store i8 %{{.+}}, i8* %{{.+}}, align 1
----------------
lei wrote:
> I don't see a run line above that uses `CHECK-BE|CHECK-LE`.  Since all these look the same, did you mean to use the default `CHECK`?
You're right - unfortunately in the previous tests that use `CHECK-BE`/`CHECK-LE` that were committed, I did not realize that I forgot to upstream the appropriate RUN lines... If it is okay, I can add them into this patch when I update it. 

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82467