Skip to content

[Release] Build compiler-rt during Phase 1 on AIX #70672

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Nov 2, 2023

Conversation

azhan92
Copy link
Contributor

@azhan92 azhan92 commented Oct 30, 2023

Compiler-rt built-ins are needed to have a functional Phase 1 clang compiler on AIX:

CMake Error at /gsa/rtpgsa-p2/00/xlcmpbldenv/aix/cmake-3.22.0/share/cmake-3.22/Modules/CMakeTestCCompiler.cmake:69 (message):
  The C compiler

    "/scratch/alisonz/llvm/clang17.0.3-final/final/Phase1/Release/llvmCore-17.0.3-final.install/usr/local/bin/clang"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: /scratch/alisonz/llvm/clang17.0.3-final/final/Phase2/Release/llvmCore-17.0.3-final.obj/CMakeFiles/CMakeTmp
    
    Run Build Command(s):/xl_le/gsa/rtpgsa/projects/w/wyvern-environment/compilers/AIX/ninja cmTC_6ad06 && [1/2] Building C object CMakeFiles/cmTC_6ad06.dir/testCCompiler.c.o
    [2/2] Linking C executable cmTC_6ad06
    FAILED: cmTC_6ad06 
    : && /scratch/alisonz/llvm/clang17.0.3-final/final/Phase1/Release/llvmCore-17.0.3-final.install/usr/local/bin/clang  -Wl,-bnoipath  CMakeFiles/cmTC_6ad06.dir/testCCompiler.c.o -o cmTC_6ad06  -Wl,-blibpath:/usr/lib:/lib && :
    ld: 0706-005 Cannot find or open file: /scratch/alisonz/llvm/clang17.0.3-final/final/Phase1/Release/llvmCore-17.0.3-final.install/usr/local/lib/clang/17/lib/aix/libclang_rt.builtins-powerpc64.a
        ld:open(): A file or directory in the path name does not exist.
    clang: error: linker command failed with exit code 255 (use -v to see invocation)
    ninja: build stopped: subcommand failed.
    

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:47 (project)


-- Configuring incomplete, errors occurred!

This PR adds compiler-rt to the runtime_list during Phase 1 to avoid this problem.

@azhan92 azhan92 marked this pull request as ready for review October 30, 2023 17:21
@azhan92 azhan92 force-pushed the alisonz/build_runtime_AIX branch from e77457e to 0909cd2 Compare October 30, 2023 17:41
@azhan92 azhan92 force-pushed the alisonz/build_runtime_AIX branch from 8040ed9 to cdb6be2 Compare October 31, 2023 15:02
Copy link
Member

@daltenty daltenty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, with minor nit

Copy link
Contributor

@amy-kwan amy-kwan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@daltenty daltenty merged commit ddfee5d into llvm:main Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants