[PATCH] D56990: Bugfix for Replacement of tied operand of inline asm

[Eli Friedman via Phabricator via cfe-commits]

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