[PATCH] D17552: Pass -backend-option to LLVM when there is no target machine

[Yaxun Liu via cfe-commits]

yaxunl marked an inline comment as done.

================
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
----------------
Anastasia wrote:
> 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.
if we change it to x86, it has a target machine, so the test no longer reproduce the original issue.

How about add -x cl for the second command to force the source to be treated as cl source code, then it can reproduce the original issue and at the same time is valid for spir target?


http://reviews.llvm.org/D17552