Skip to content

Commit

Permalink
[OpenMP] Remove the workaround of passing "-x assembler-with-cpp" man…
Browse files Browse the repository at this point in the history
…ually

By building the assembly with language ASM now (since
4072c8a and
cbaa359), this shouldn't be
needed any longer.

Differential Revision: https://reviews.llvm.org/D150701
  • Loading branch information
mstorsjo committed Jul 7, 2023
1 parent 852fe30 commit f105c1d
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 6 deletions.
1 change: 0 additions & 1 deletion openmp/runtime/cmake/LibompHandleFlags.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,6 @@ endfunction()
# Assembler flags
function(libomp_get_asmflags asmflags)
set(asmflags_local)
libomp_append(asmflags_local "-x assembler-with-cpp" LIBOMP_HAVE_X_ASSEMBLER_WITH_CPP_FLAG)
# Architectural assembler flags
if(${IA32})
if(CMAKE_SIZEOF_VOID_P EQUAL 8)
Expand Down
5 changes: 0 additions & 5 deletions openmp/runtime/cmake/config-ix.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -96,11 +96,6 @@ if(WIN32)
)
endforeach()
endforeach()
else()
# It is difficult to create a dummy assembly file that compiles into an
# executable for every architecture and then check the C compiler to
# see if -x assembler-with-cpp exists and works, so we assume it does for non-Windows.
set(LIBOMP_HAVE_X_ASSEMBLER_WITH_CPP_FLAG TRUE)
endif()
if(${LIBOMP_FORTRAN_MODULES})
libomp_check_fortran_flag(-m32 LIBOMP_HAVE_M32_FORTRAN_FLAG)
Expand Down

0 comments on commit f105c1d

Please sign in to comment.