[clang] [flang] [flang][nfc] Refactor linker invocation logic (PR #75534)

2023-12-15 Thread Pete Steinfeld via cfe-commits
https://github.com/psteinfeld commented: After this change, I can no longer use flang-new to create an executable program. When I try, I get the following output: ``` [psteinfeld@ice4 build]$ flang-new hello.f90 /opt/rh/gcc-toolset-12/root/usr/lib/gcc/x86_64-redhat-linux/12/../../../../bin/ld:

[clang] [flang][nfc] Refactor linker invocation logic (PR #75648)

2023-12-15 Thread Pete Steinfeld via cfe-commits
https://github.com/psteinfeld approved this pull request. All looks good. Thanks for the quick fix! https://github.com/llvm/llvm-project/pull/75648 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo

[clang] [flang] [flang][Driver] Let the linker fail on multiple definitions of main() (PR #73124)

2023-11-28 Thread Pete Steinfeld via cfe-commits
psteinfeld wrote: @mjklemm, after this change was integrated, the test Driver/ctofortran no longer succeeds. This test gets run when you call `make check-flang`. I'm not sure why the pre- and post-build checks did not run check-flang. https://github.com/llvm/llvm-project/pull/73124 _

[clang] [flang] [flang][Driver] Let the linker fail on multiple definitions of main() (PR #73124)

2023-11-28 Thread Pete Steinfeld via cfe-commits
psteinfeld wrote: > @mjklemm, after this change was integrated, the test Driver/ctofortran no > longer succeeds. This test gets run when you call `make check-flang`. I'm not > sure why the pre- and post-build checks did not run check-flang. I've submitted #73738 to fix this. https://github.co

[clang] [flang] [flang] Fixes for LIT testing of FLANG_RUNTIME_F128_MATH_LIB build. (PR #82832)

2024-02-26 Thread Pete Steinfeld via cfe-commits
https://github.com/psteinfeld approved this pull request. All builds and tests correctly and looks good. https://github.com/llvm/llvm-project/pull/82832 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/list

[clang-tools-extra] [llvm] [flang] [clang] [flang] Add EXECUTE_COMMAND_LINE runtime and lowering intrinsics implementation (PR #74077)

2024-01-12 Thread Pete Steinfeld via cfe-commits
psteinfeld wrote: See issue #77975. https://github.com/llvm/llvm-project/pull/74077 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[llvm] [clang] [flang] [clang-tools-extra] [flang] Add EXECUTE_COMMAND_LINE runtime and lowering intrinsics implementation (PR #74077)

2024-01-12 Thread Pete Steinfeld via cfe-commits
psteinfeld wrote: See issue #77984 https://github.com/llvm/llvm-project/pull/74077 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-tools-extra] [flang] [llvm] [flang] Add EXECUTE_COMMAND_LINE runtime and lowering intrinsics implementation (PR #74077)

2024-01-12 Thread Pete Steinfeld via cfe-commits
psteinfeld wrote: See issue #77979. https://github.com/llvm/llvm-project/pull/74077 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang] [flang] [libc] [compiler-rt] [llvm] [flang] use setsid to assign the child to prevent zombie as it will be clean up by init process (PR #77944)

2024-01-18 Thread Pete Steinfeld via cfe-commits
https://github.com/psteinfeld approved this pull request. All builds and tests correctly and looks good. https://github.com/llvm/llvm-project/pull/77944 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/list

[clang] [openmp] Revert "[Libomp] Place generated OpenMP headers into build resource d… (PR #88083)

2024-04-08 Thread Pete Steinfeld via cfe-commits
https://github.com/psteinfeld created https://github.com/llvm/llvm-project/pull/88083 …irectory (#88007)" This reverts commit 8671429151d5e67d3f21a737809953ae8bdfbfde. This commit broke the flang build, so I'm reverting it. See the comments in merge request #88007 for more information. >Fro

[clang] [openmp] [Libomp] Place generated OpenMP headers into build resource directory (PR #88007)

2024-04-08 Thread Pete Steinfeld via cfe-commits
psteinfeld wrote: Hey, @jhuber. This change broke the flang build. When I revert the change, everything builds cleanly. So I reverted it in merge request After this change was added, I started getting messages like: ``` [4/205] Building CXX object projects/openmp/libomptarget/plugins-nex

[clang] [openmp] Revert "[Libomp] Place generated OpenMP headers into build resource d… (PR #88083)

2024-04-08 Thread Pete Steinfeld via cfe-commits
https://github.com/psteinfeld closed https://github.com/llvm/llvm-project/pull/88083 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [openmp] [Libomp] Place generated OpenMP headers into build resource directory (PR #88007)

2024-04-09 Thread Pete Steinfeld via cfe-commits
psteinfeld wrote: I should have mentioned, I'm building with GCC 9.3.0. Most of my team uses this compiler. https://github.com/llvm/llvm-project/pull/88007 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mail