[PATCH] D111566: [SYCL] Fix function pointer address space

Dylan McKay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 1 01:18:03 PST 2021


dylanmckay accepted this revision.
dylanmckay added a comment.

By the way, as this has already been approved by one, and you rightly applied the "speak now or forever hold your peace" principle re. OpenCL, and this clearly works better from my point of view than the old code, I wouldn't want to prevent you from committing this. **I have no objections to merging this in its current state**. If you do merge it as-is though it would be nice if a follow up PR moves the logic into `getTargetAddressSpace()` where I think it better belongs


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

https://reviews.llvm.org/D111566



More information about the cfe-commits mailing list