This is an archive of the discontinued LLVM Phabricator instance.

Fixes gfortran compilation failures caused by missing 'break' in the fix for PR22234 committed in r227409.
ClosedPublic

Authored by tra on Jan 30 2015, 10:15 AM.

Details

Reviewers
echristo
Summary

Due to missing break -fcompile-only was always passed to fortran
compiler which in turn didn't produce any output which caused
assembler to fail.

Diff Detail

Event Timeline

tra updated this revision to Diff 19045.Jan 30 2015, 10:15 AM
tra retitled this revision from to Fixes gfortran compilation failures caused by missing 'break' in the fix for PR22234 committed in r227409..
tra updated this object.
tra edited the test plan for this revision. (Show Details)
tra added a subscriber: Unknown Object (MLST).
echristo accepted this revision.Jan 30 2015, 10:20 AM
echristo edited edge metadata.

LGTM, sorry I missed this in review. I'll commit shortly.

Thanks!

-eric

This revision is now accepted and ready to land.Jan 30 2015, 10:20 AM
echristo closed this revision.Jan 30 2015, 10:24 AM

Applied as:

dzur:~/sources/llvm/tools/clang> git svn dcommit
Committing to https://llvm.org/svn/llvm-project/cfe/trunk ...
M lib/Driver/Tools.cpp
M test/Driver/gfortran.f90
Committed r227591