This is an archive of the discontinued LLVM Phabricator instance.

[flang] Warn for the limit on name length
ClosedPublic

Authored by peixin on May 11 2022, 3:48 AM.

Details

Summary

As fortran 2018 C601, the maximum length of a name is 63 characters.

Diff Detail

Event Timeline

peixin created this revision.May 11 2022, 3:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 11 2022, 3:48 AM
Herald added a subscriber: jdoerfert. · View Herald Transcript
peixin requested review of this revision.May 11 2022, 3:48 AM
klausler added inline comments.May 11 2022, 8:30 AM
flang/lib/Semantics/check-declarations.cpp
204

This message should be a portability warning, not a fatal error.

peixin updated this revision to Diff 428884.May 12 2022, 2:19 AM
peixin retitled this revision from [flang] Enforce limit on name length to [flang] Warn for the limit on name length.

Change errors into warnings.

klausler added inline comments.May 12 2022, 8:48 AM
flang/lib/Semantics/check-declarations.cpp
204

Portability warnings are "xxxx"_port_en_US.

peixin updated this revision to Diff 428974.May 12 2022, 9:31 AM

Change into the portability warning.

peixin added inline comments.May 12 2022, 9:38 AM
flang/lib/Semantics/check-declarations.cpp
204

Thanks. Fixed. I don't find the test cases for this kind of warnings, so I remove the test case.

klausler accepted this revision.May 12 2022, 9:42 AM
This revision is now accepted and ready to land.May 12 2022, 9:42 AM
This revision was automatically updated to reflect the committed changes.