[PATCH] D20339: Update clang for D20260
Peter Collingbourne via cfe-commits
cfe-commits at lists.llvm.org
Tue May 17 16:24:34 PDT 2016
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
More information about the cfe-commits
mailing list