[PATCH] D33597: [OpenCL] Fix pipe size in TypeInfo.

Yaxun Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 26 07:33:20 PDT 2017


yaxunl added inline comments.


================
Comment at: lib/AST/ASTContext.cpp:1942
   case Type::Pipe: {
-    TypeInfo Info = getTypeInfo(cast<PipeType>(T)->getElementType());
-    Width = Info.Width;
-    Align = Info.Align;
+    Width = Target->getPointerWidth(0);
+    Align = Target->getPointerAlign(0);
----------------
pipe is in global address space. shouldn't it be

```
Width = Target->getPointerWidth(getTargetAddressSpace(LangAS::opencl_global));
```
the same with align.


https://reviews.llvm.org/D33597





More information about the cfe-commits mailing list