pcc added inline comments.

================
Comment at: test/CodeGenOpenCL/constant-addr-space-globals.cl:3
@@ -2,3 +2,3 @@
 
-// CHECK: @array = addrspace({{[0-9]+}}) constant
+// CHECK: @array = local_unnamed_addr addrspace({{[0-9]+}}) constant
 __constant float array[2] = {0.0f, 1.0f};
----------------
rsmith wrote:
> Where does this come from? It doesn't look like Clang adds this, and this 
> test does not appear to enable any optimizations.
OpenCL enables optimizations by default in the frontend, see 
`getOptimizationLevel` in `lib/Frontend/CompilerInvocation.cpp`.

It does seem a little weird that we're doing this in the frontend, this should 
probably be a driver feature. I must apologise for me-from-5-years-ago who 
implemented this in r120876.

In the meantime, maybe we should be adding `-cl-opt-disable` to the OpenCL 
codegen tests.


http://reviews.llvm.org/D20339



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to