Page MenuHomePhabricator

[flang][nfc] Port 2 tests to use the new driver when enabled
ClosedPublic

Authored by awarzynski on Apr 16 2021, 8:33 AM.

Details

Summary

This is similar to https://reviews.llvm.org/D100309, i.e. %f18 is
replaced with %flang_new.

resolve105.f90 wasn't in tree when D100309 was worked on, so it's
updated here instead.

label14.f90 requires -fsyntax-only. I didn't notice that when
submitting D100309, hence updating it now instead. -fsyntax-only is
required to prevent %f18 from calling an external compiler (which then
fails and returns a non-zero exit code).

Diff Detail

Event Timeline

awarzynski created this revision.Apr 16 2021, 8:33 AM
awarzynski requested review of this revision.Apr 16 2021, 8:33 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 16 2021, 8:33 AM
awarzynski added a project: Restricted Project.

@Leporacanthicus , since you reviewed https://reviews.llvm.org/D100309, would you mind checking this one as well?

DavidTruby accepted this revision.Apr 20 2021, 3:05 AM
DavidTruby added a subscriber: DavidTruby.

LGTM

This revision is now accepted and ready to land.Apr 20 2021, 3:05 AM
SouraVX accepted this revision.Apr 20 2021, 3:38 AM