================
@@ -582,6 +582,10 @@ class Driver {
/// @name Helper Methods
/// @{
+ /// getSYCLDeviceTriple - Returns the SYCL device triple for the
+ /// specified ArchType.
+ llvm::Triple getSYCLDeviceTriple(StringRef TargetArch = "spir64") const;
----------------
tahonermann wrote:
It doesn't look like `getSYCLDeviceTriple()` needs to be exposed in the header.
Can this just be a `static` function in `Driver.cpp`? There are similar
`static` functions there for CUDA and HIP; see `getNVIDIAOffloadTargetTriple()`
and `getHIPOffloadTargetTriple()`. I think this should follow that precedent if
there isn't a good reason to deviate.
https://github.com/llvm/llvm-project/pull/107493
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
