Skip to content

fixes #803 and #772 - #804

Merged
awvwgk merged 2 commits into
fortran-lang:mainfrom
urbanjost:main
Nov 27, 2022
Merged

fixes #803 and #772#804
awvwgk merged 2 commits into
fortran-lang:mainfrom
urbanjost:main

Conversation

@urbanjost

Copy link
Copy Markdown
Contributor

Fixes a problem where the first preprocessor macro definition is not always preceded by a space.

See
#772

or try to do

ifort -cpp testit.f90

      program testit
      integer :: unix=1
      integer :: linux=1
      end program testit

@awvwgk awvwgk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me. Thanks for sharing, John.

@urbanjost

Copy link
Copy Markdown
Contributor Author

Well, thanks for reviewing. I hit the bug in -D with actual usage so it is self-serving on my part :>.

@awvwgk
awvwgk merged commit e781a77 into fortran-lang:main Nov 27, 2022
@perazz perazz mentioned this pull request Apr 19, 2023
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.

Missing space before -D when specifying preprocessing from the fpm.toml file Bugs encountered when trying to bootstrap with intel ifort 2021.7

2 participants