Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Internal Compiler Error when compiling arithmetic if code with -i8 flag #844

Open
kiranchandramohan opened this issue Dec 11, 2019 · 0 comments

Comments

@kiranchandramohan
Copy link
Collaborator

kiranchandramohan commented Dec 11, 2019

Code with arithmetic if when compiled using the -i8 flag shows an internal compiler error.
F90-F-0000-Internal compiler error. addili: illegal IL_TYPE(opc) 0

IF (J) 85,85,105
85 I = J
105 I = J
END 

This is a regression introduced (a4012a9) when adding half-precision support. The issue happens because the aif array size was increased from 4 to 5 and the index for integer8 increased from 3 to 4 but the entry for half-precision was not added. Hence the index for integer8 pointing to an incorrect location.
Relevant file: tools/flang2/flang2exe/exp_ftn.cpp

Is the correct fix to

  1. Add the entry for half precision. Currently not able to test it since parser does not seem to be taking in real*2.
    or
  2. Revert to the original array size and index for integer*8.

I can make Pull Requests for both 1 and 2 if necessary.

edoapra added a commit to nwchemgit/nwchem that referenced this issue Jan 23, 2020
edoapra added a commit to nwchemgit/nwchem that referenced this issue Jan 23, 2020
edoapra added a commit to nwchemgit/nwchem that referenced this issue Feb 13, 2020
edoapra added a commit to nwchemgit/nwchem that referenced this issue Feb 13, 2020
edoapra added a commit to nwchemgit/nwchem that referenced this issue Feb 13, 2020
edoapra added a commit to nwchemgit/nwchem that referenced this issue Feb 15, 2020
edoapra added a commit to nwchemgit/nwchem that referenced this issue Feb 15, 2020
edoapra added a commit to nwchemgit/nwchem that referenced this issue Feb 15, 2020
edoapra added a commit to nwchemgit/nwchem that referenced this issue Mar 16, 2020
edoapra added a commit to nwchemgit/nwchem that referenced this issue Jul 8, 2020
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

No branches or pull requests

1 participant