topic Re: Bug in MKLConfig.cmake in Intel® oneAPI Math Kernel Library

Bug in MKLConfig.cmake

JoeTodd — Wed, 02 Mar 2022 18:36:26 GMT

In MKLConfig.cmake, the compiler matching regexes:

if(CMAKE_CXX_COMPILER MATCHES ".*dpcpp[.exe]*") set(DPCPP_COMPILER ON) endif() if(CMAKE_C_COMPILER MATCHES ".*clang*") set(CLANG_COMPILER ON) endif()

aren't doing what they ought to. An optional .exe extension should be represented as:

".*dpcpp(\\.exe)?$"

and I believe that ".*clang*" would match any compiler path with "clan" somewhere in it.

Re: Bug in MKLConfig.cmake

Viraj_M_Intel — Wed, 02 Mar 2022 17:31:42 GMT

Hello JoeTodd,

Can you please share what you are trying to do? Some kind of an example will help us look into the problem in detail.

Re: Bug in MKLConfig.cmake

JoeTodd — Wed, 02 Mar 2022 18:37:27 GMT

Hi Viraj,

I'm not really trying to do anything - I just noticed this bug & thought I ought to report it.

Joe

Re: Bug in MKLConfig.cmake

ShanmukhS_Intel — Tue, 29 Mar 2022 11:18:10 GMT

Hello JoeTodd,

Thank you for posting on Intel Communities.

Based on the revised snippet shared by you(".*dpcpp(\\.exe)?$"), you mean a must .exe to be expected after dpcpp and it should not accept any character. However, as per the existing exe, (.*dpcpp[.exe]*) is accepts any of the mentioned characters within the braces. Could you please let me know if my understanding correct?

Best Regards,

Shanmukh.SS

Re:Bug in MKLConfig.cmake

ShanmukhS_Intel — Thu, 17 Mar 2022 12:33:26 GMT

Hi,

Reminder:

Could you please let us know if our understanding is correct, so that we could proceed further on this case.

Best Regards,

Shanmukh.SS

Re: Re:Bug in MKLConfig.cmake

JoeTodd — Tue, 22 Mar 2022 11:01:57 GMT

Square brackets in a regular expression indicate a set of acceptable characters, *not* a group, which is what is intended here. For example:

"(\\.exe)?" would match zero or one instances of the string ".exe" at the end of a path - in other words it defines an optional .exe suffix

"[.exe]*" defines a set of characters ".", "e", "x", "e", and states that zero or more of these characters is acceptable. Furthermore, because the period is not escaped with \\, the "." character actually matches any character. This means that "[.exe]*" is actually equivalent to "[.]*". And given that the expression "[.exe]*" isn't followed by any other character/requirement, the entire "[.exe]*" is actually redundant, because it says "we are looking for zero or more of any character".

Re:Bug in MKLConfig.cmake

ShanmukhS_Intel — Wed, 30 Mar 2022 06:16:23 GMT

Hi,

Thank you for sharing the concern. We are working on this internally. We'll get back to you soon with an update.

Best Regards,

Shanmukh.SS

Re:Bug in MKLConfig.cmake

Khang_N_Intel — Thu, 02 Jun 2022 00:59:52 GMT

Hi Joe,

The fix will be in oneMKL 2022.2. We will let you know when this release is launched.

Best regards,

Khang