azabaznov added inline comments.

================
Comment at: clang/lib/AST/ExprConstant.cpp:9554
+
+    const auto &OpenCLFeaturesMap =
+        Info.Ctx.getTargetInfo().getSupportedOpenCLOpts();
----------------
azabaznov wrote:
> Anastasia wrote:
> > What test case covers this change? It feels like something we should try to 
> > diagnose earlier...
> The test case which exactly was added. Since blocks in constant address space 
> are disallowed at this point, we can treat all other blocks with no captures 
> not as constant expressions - it will make CodeGen generate block  literal in 
> local scope for blocks with no captures. See `buildGlobalBlock `and 
> `CodeGenModule::GetAddrOfGlobalBlock` for details.
I mean - //blocks_no_global_literal.cl// - that's exactly the case which covers 
it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112230/new/

https://reviews.llvm.org/D112230

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

Reply via email to