Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(17)

Unified Diff: src/IceCompiler.cpp

Issue 1132883003: Change a build-attribute REQUIRES check to use supported not requested. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | tests_lit/llvm2ice_tests/addr-opt-multi-def-var.ll » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/IceCompiler.cpp
diff --git a/src/IceCompiler.cpp b/src/IceCompiler.cpp
index 7c8b791969e63feafdac29b17421876e2640befc..256a844b28cabb4662b523f7d9b3ff0239738e85 100644
--- a/src/IceCompiler.cpp
+++ b/src/IceCompiler.cpp
@@ -48,9 +48,6 @@ struct {
// 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
jvoung (off chromium) 2015/05/15 23:50:30 Hmm, clang-format seems to force the comment to st
Jim Stichnoth 2015/05/15 23:52:06 No problem either way, just want to avoid the "nex
#define SUBZERO_TARGET(TARGET) *Stream << "target_" #TARGET << "\n";
#include "llvm/Config/SZTargets.def"
« no previous file with comments | « no previous file | tests_lit/llvm2ice_tests/addr-opt-multi-def-var.ll » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698