Change a build-attribute REQUIRES check to use supported not requested.

Previously it would print both the targets compiled-in and the target requested on the commandline, but we really only care about what's compiled-in. BUG=none R=stichnot@chromium.org Review URL: https://codereview.chromium.org/1132883003

Change a build-attribute REQUIRES check to use supported not requested.
33492e76 · Jan Voung · 76bb0bec · 33492e76 · 33492e76
Commit 33492e76 authored May 15, 2015 by Jan Voung
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 5 deletions

IceCompiler.cpp src/IceCompiler.cpp +1 -4

addr-opt-multi-def-var.ll tests_lit/llvm2ice_tests/addr-opt-multi-def-var.ll +1 -1

No files found.
--- a/src/IceCompiler.cpp
+++ b/src/IceCompiler.cpp
@@ -47,11 +47,8 @@ struct {
 // Validates values of build attributes. Prints them to Stream if
 // Stream is non-null.
 void ValidateAndGenerateBuildAttributes(const ClFlags &Flags, Ostream *Stream) {
+  // List the supported targets.
  if (Stream) {
-    // List the requested target.
-    *Stream << Flags.getTargetArch() << "\n";
-
-// List the supported targets.
 #define SUBZERO_TARGET(TARGET) *Stream << "target_" #TARGET << "\n";
 #include "llvm/Config/SZTargets.def"
  }

--- a/tests_lit/llvm2ice_tests/addr-opt-multi-def-var.ll
+++ b/tests_lit/llvm2ice_tests/addr-opt-multi-def-var.ll
@@ -5,7 +5,7 @@
 ; This test depends to some degree on the stability of "--verbose
 ; addropt" output format.

-; REQUIRES: x86-32
+; REQUIRES: target_X8632
 ; REQUIRES: allow_dump
 ; RUN: %p2i -i %s --args -O2 --verbose addropt | FileCheck %s