[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
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?


More information about the cfe-commits mailing list