chapuni added inline comments.

================
Comment at: cfe/trunk/lib/Basic/TargetInfo.cpp:351
 
+LangAS::ID TargetInfo::getOpenCLTypeAddrSpace(const Type *T) const {
+  auto BT = dyn_cast<BuiltinType>(T);
----------------
Excuse me for old commit, I think it might be layering violation.
Could we avoid using AST/Type here?


Repository:
  rL LLVM

https://reviews.llvm.org/D33989



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D33989: [OpenCL]... NAKAMURA Takumi via Phabricator via cfe-commits

Reply via email to