[PATCH] Add HSAIL support

Tom Stellard thomas.stellard at amd.com
Mon Jun 22 06:58:16 PDT 2015


================
Comment at: lib/Basic/Targets.cpp:5416
@@ +5415,3 @@
+    HSAIL_NONE,
+    HSAIL_KAVERI
+  } Device;
----------------
Do we need an HSAIL_CARRIZO target too?

================
Comment at: lib/Basic/Targets.cpp:5482
@@ +5481,3 @@
+    if (FullProfile && Opts.OpenCL) {
+      Builder.defineMacro("cl_khr_fp64");
+    }
----------------
There should be more defines here, like in this patch: http://reviews.llvm.org/D10601

http://reviews.llvm.org/D10586

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list