This is an archive of the discontinued LLVM Phabricator instance.

[flang][driver] Modify the existing test cases that use -Mstandard in f18, to use -pedantic and %flang_fc1 to share with the new driver
ClosedPublic

Authored by arnamoy10 on Mar 29 2021, 10:07 AM.

Details

Summary

[flang][driver] Modify the existing test cases that use -Mstandard in f18, to use -pedantic and %flang_fc1 to share with the new driver

One of the test cases (flang/test/Semantics/dosemantics03.f90) fails currently when tested against the new driver as -Werror is used in addition to -pedantic in that test. When -Werror lands (https://reviews.llvm.org/D98657) for the new driver, this error will go away.

Diff Detail

Event Timeline

arnamoy10 created this revision.Mar 29 2021, 10:07 AM
arnamoy10 requested review of this revision.Mar 29 2021, 10:07 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 29 2021, 10:07 AM
arnamoy10 edited the summary of this revision. (Show Details)Mar 29 2021, 10:08 AM
arnamoy10 edited the summary of this revision. (Show Details)Mar 29 2021, 12:45 PM
awarzynski accepted this revision.Mar 30 2021, 9:27 AM

Thank you for working on this! I would be tempted to make this patch depend on https://reviews.llvm.org/D98657. It's up to you.

[nit] I think that it's worth shortening the commit summary (i.e. the first line). Perhaps this: [flang][driver] Port tests to use -pedantic` instead of -Mstandard. Shorter messages do make git log easier to navigate. You could move the contents of your top line to the summary.

As the above are nits, accepting as is!

This revision is now accepted and ready to land.Mar 30 2021, 9:27 AM