Anastasia accepted this revision. Anastasia added a comment. LGTM, apart from small remark on the test!
================ Comment at: test/Frontend/backend-option.c:2 @@ +1,3 @@ +// RUN: %clang_cc1 %s -emit-llvm -backend-option -time-passes -o - 2>&1 | FileCheck %s +// RUN: %clang_cc1 %s -emit-llvm -backend-option -time-passes -o - -triple spir-unknown-unknown 2>&1 | FileCheck %s +// CHECK: Pass execution timing report ---------------- I think conceptually it's not correct to pass spir target with C module, although it doesn't make any difference here for this test. I would still change it to some x86 one. http://reviews.llvm.org/D17552 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
