echristo accepted this revision. echristo added a comment. This revision is now accepted and ready to land.
This is fine. One inline comment. -eric ================ Comment at: test/CodeGen/tbm-builtins.c:2-3 @@ -1,3 +1,4 @@ // RUN: %clang_cc1 %s -O3 -triple=x86_64-unknown-unknown -target-feature +tbm -emit-llvm -o - | FileCheck %s +// REQUIRES: x86-registered-target // FIXME: The code generation checks for add/sub and/or are depending on the optimizer. ---------------- Go ahead and put the REQUIRES underneath the FIXME and add a note to the effect that the REQUIRES can be removed when the FIXME is complete. http://reviews.llvm.org/D14887 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits