Skip to content

Commit 4b6597f

Browse files
committed
Make flang driver stuff work on macOS
6bf5580 added special-case code for TY_PP_Fortran to ToolChain::LookupTypeForExtension(), but Darwin::LookupTypeForExtension() overrode that method without calling the superclass implementation. Make it call the superclass implementation to fix things. Differential Revision: https://reviews.llvm.org/D69636
1 parent a780b94 commit 4b6597f

File tree

2 files changed

+1
-5
lines changed

2 files changed

+1
-5
lines changed

clang/lib/Driver/ToolChains/Darwin.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -737,7 +737,7 @@ Darwin::Darwin(const Driver &D, const llvm::Triple &Triple, const ArgList &Args)
737737
CudaInstallation(D, Triple, Args) {}
738738

739739
types::ID MachO::LookupTypeForExtension(StringRef Ext) const {
740-
types::ID Ty = types::lookupTypeForExtension(Ext);
740+
types::ID Ty = ToolChain::lookupTypeForExtension(Ext);
741741

742742
// Darwin always preprocesses assembly files (unless -x is used explicitly).
743743
if (Ty == types::TY_PP_Asm)

clang/test/Driver/flang/flang.f90

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,3 @@
1-
! D63607 made mac builders unhappy by failing this test, and it isn't
2-
! yet obvious why. Mark as unsupported as a temporary measure.
3-
! UNSUPPORTED: darwin
4-
51
! Check that flang -fc1 is invoked when in --driver-mode=flang.
62

73
! This is a copy of flang_ucase.F90 because the driver has logic in it which

0 commit comments

Comments
 (0)