Skip to content

Conversation

jvdp1
Copy link
Member

@jvdp1 jvdp1 commented May 30, 2020

Mentioned by @MarDiehl in #205

ifort message:

     function diag_cdp_mat(A) result(res)
---------------^
/home/jvandenp/stdlib/build/src/stdlib_experimental_linalg_diag.f90(350): error #6645: The name of the module procedure conflicts with a name in the encompassing scoping unit.   [DIAG_CQP_MA
T]
      function diag_cqp_mat(A) result(res)
---------------^
/home/jvandenp/stdlib/build/src/stdlib_experimental_linalg_diag.f90(358): error #6645: The name of the module procedure conflicts with a name in the encompassing scoping unit.   [DIAG_IINT8_
MAT]

ifort message:
```
     function diag_cdp_mat(A) result(res)
---------------^
/home/jvandenp/stdlib/build/src/stdlib_experimental_linalg_diag.f90(350): error #6645: The name of the module procedure conflicts with a name in the encompassing scoping unit.   [DIAG_CQP_MA
T]
      function diag_cqp_mat(A) result(res)
---------------^
/home/jvandenp/stdlib/build/src/stdlib_experimental_linalg_diag.f90(358): error #6645: The name of the module procedure conflicts with a name in the encompassing scoping unit.   [DIAG_IINT8_
MAT]
```
Copy link
Member

@ivan-pi ivan-pi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yes, I noticed I was making this error later in some private projects, but it didn't show up in the CI since we only test with gfortran.

@jvdp1
Copy link
Member Author

jvdp1 commented May 31, 2020

Since it is a correction, I'll merge.

@jvdp1 jvdp1 merged commit ff99e00 into fortran-lang:master May 31, 2020
@jvdp1 jvdp1 deleted the fix_intel_diag branch May 31, 2020 14:35
@jvdp1
Copy link
Member Author

jvdp1 commented May 31, 2020

Fixes one of the issues in #205

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants