efriedma added inline comments.

================
Comment at: test/Sema/inline-asm-x86-constraint.c:2
+// REQUIRES: x86-registered-target
+// RUN: %clang_cc1  %s -o %t
+typedef unsigned char __attribute__((vector_size(8))) _m64u8;
----------------
Test belongs in test/CodeGen.  Please use -emit-llvm and FileCheck the IR, 
instead of checking how the backend handles it.  Shouldn't require 
x86-registered-target (the x86 frontend bits are always compiled in).

Actually, you could probably stick the function into the existing test 
test/CodeGen/asm-inout.c, which has another similar test.


Repository:
  rC Clang

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

https://reviews.llvm.org/D56990



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to