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

Anastasia Stulova via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 22 12:19:12 PDT 2016


Anastasia added inline comments.

================
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
----------------
yaxunl wrote:
> 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?
Sure!


http://reviews.llvm.org/D17552





More information about the cfe-commits mailing list