This is an archive of the discontinued LLVM Phabricator instance.

Fixed workaround made in https://reviews.llvm.org/D51694
ClosedPublic

Authored by AndreyChurbanov on Oct 1 2018, 2:59 AM.

Details

Summary

Kelvin Li suggested solution for using long name with long kind name in the Fortran header file - omit optional "kind=" part of the kind-selector.
This patch fixes the workarount I used earlier (use shorter kind name that is currently has the same value, but theoretically can be changed somewhen in future).

Also removed the "kind=" for surrounded constants (not sure if we should delete it everywhere??).

Diff Detail

Repository
rL LLVM

Event Timeline

AndreyChurbanov created this revision.Oct 1 2018, 2:59 AM
hbae accepted this revision.Oct 1 2018, 6:57 AM

LGTM.

This revision is now accepted and ready to land.Oct 1 2018, 6:57 AM
This revision was automatically updated to reflect the committed changes.