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

src/IceCompiler.cpp

 //===- subzero/src/IceCompiler.cpp - Driver for bitcode translation -------===//
 //
 //                        The Subzero Code Generator
 //
 // This file is distributed under the University of Illinois Open Source
 // License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
 //
 // This file defines a driver for translating PNaCl bitcode into native code.
@@ skipping 30 matching lines @@
                                   {"llvm_cl", ALLOW_LLVM_CL},
                                   {"llvm_ir", ALLOW_LLVM_IR},
                                   {"llvm_ir_as_input", ALLOW_LLVM_IR_AS_INPUT},
                                   {"minimal_build", ALLOW_MINIMAL_BUILD},
                                   {"browser_mode", PNACL_BROWSER_TRANSLATOR}};
 
 // Validates values of build attributes. Prints them to Stream if
 // Stream is non-null.
 void ValidateAndGenerateBuildAttributes(const ClFlags &Flags, Ostream *Stream) {
   if (Stream) {
-    // List the requested target.
-    *Stream << Flags.getTargetArch() << "\n";
-
 // List the supported targets.

[Jim Stichnoth, 2015/05/15 23:21:50]

This comment's indentation is weird (but was also weird before) - make sure
clang-format agrees with it?

[jvoung (off chromium), 2015/05/15 23:50:30]

Hmm, clang-format seems to force the comment to stick with the #define indent
level, and also forces the #define to be flushed left.

I moved the comment before the if (Stream), as one weird trick to get better
indenting.

[Jim Stichnoth, 2015/05/15 23:52:06]

No problem either way, just want to avoid the "next guy" getting an unexpected
formatting change from clang-format.

 #define SUBZERO_TARGET(TARGET) *Stream << "target_" #TARGET << "\n";
 #include "llvm/Config/SZTargets.def"
   }
 
   for (size_t i = 0; i < llvm::array_lengthof(ConditionalBuildAttributes);
        ++i) {
     switch (ConditionalBuildAttributes[i].FlagValue) {
     case 0:
       if (Stream)
         *Stream << "no_" << ConditionalBuildAttributes[i].FlagName << "\n";
@@ skipping 96 matching lines @@
     Ctx.dumpTimers();
   if (Ctx.getFlags().getTimeEachFunction()) {
     const bool DumpCumulative = false;
     Ctx.dumpTimers(GlobalContext::TSK_Funcs, DumpCumulative);
   }
   const bool FinalStats = true;
   Ctx.dumpStats("_FINAL_", FinalStats);
 }
 
 } // end of namespace Ice