[PATCH] D17552: Pass -backend-option to LLVM when there is no target machine
Yaxun Liu via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 22 11:46:20 PDT 2016
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
> 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?
More information about the cfe-commits