[PATCH] D24735: Triple: Add opencl environment type

Anastasia Stulova via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 27 10:14:10 PDT 2016


Anastasia added inline comments.

================
Comment at: include/llvm/ADT/Triple.h:194
@@ -193,2 +193,3 @@
     CoreCLR,
-    LastEnvironmentType = CoreCLR
+    OpenCL,
+    LastEnvironmentType = OpenCL
----------------
tstellarAMD wrote:
> Anastasia wrote:
> > What would be the difference to AMDOpenCL item above?
> AMDOpenCL was used with the X86 arch only.  I wanted to add something that would be more extensible and would make sense for other vendors.
I see. Could AMDOpenCL be replaced by OpenCL as more generic item then?


Repository:
  rL LLVM

https://reviews.llvm.org/D24735





More information about the llvm-commits mailing list