-
Notifications
You must be signed in to change notification settings - Fork 31
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
Port Classic Flang changes to LLVM 16 #159
Port Classic Flang changes to LLVM 16 #159
Commits on Jul 4, 2023
-
This commit cherry-picks 427c0e8 from the release_14x branch, which is a combination of the following legacy patches: Commit 82b38d2: [ClassicFlang] Port release_90 changes from flang-compiler/llvm Cherry-picked commit 2085211cfcca70411dc63f0d08763facc8a02090 by Eric Schweitz, resolved merge conflicts, fixed build failures (e.g. adapted CGDebugInfo.cpp to the new API), and fixed the DIGlobalVariable unit tests, which have been broken since commit edfad65eebdf045b050f37380b6b61d673513982. Commit 885dd87e5fdc: [DebugInfo] Removal of DIFortranArrayType and DIFortranSubrange These extensions are no more required after merge of below PR. Commit 5c9b2e0867d5: Modification to incorporate comments from @bryanpkc. Commit 822de2c: [AsmPrinter] Fix redundant names and types A bug was introduced in 82b38d2 while cherry-picking a DIGlobalVariable-related patch. Commit 45a70a8: Port driver changes from release/11x Cherry-picked c51f89679135f84675f492d560ec5535c2000cfe by Varun Jayathirtha from release_90, and resolved merge conflicts. To avoid conflicts with the new Flang, lib/Driver/ToolChains/Flang.{cpp,h} have been renamed to ClassicFlang.{cpp,h}, and the ENABLE_CLASSIC_FLANG macro is introduced to select which incarnation of Flang to build. The macro is set by running CMake with -DLLVM_ENABLE_CLASSIC_FLANG. After merge with LLVM 11: Move flang options to the end of the definitions list. Commit a9a8036: Port Classic Flang to LLVM 13 File changes to TargetLibraryInfo and DebugLocEntry to adapt to the code from release/13.x and make it work. Comment out the changes due a segmentation fault, code need to be reviewed properly once all commits are in Commit fe989b7: Fix -fveclib=PGMATH The use of #ifdef in include/clang/Driver/Options.td was incorrect and unsupported. As a result -fveclib=PGMATH was silently ignored, and in LLVM 12, it causes the invocation to fail. This patch unguards the option so that it is parsed correctly, but lets the FLANG_LLVM_EXTENSIONS macro continue to toggle the feature. Commit 7c224ae: Fix use of classic Flang as preprocessor Commit 8403c83: Merge FLANG_LLVM_EXTENSIONS macro into ENABLE_CLASSIC_FLANG Commit 486741e: Fix test failures when in classic Flang mode Add a new lit feature tag "classic_flang" to select which tests can or cannot be run when the driver is built for classic Flang. Handle LLVM_ENABLE_CLASSIC_FLANG in llvm/cmake/modules/HandleLLVMOptions.cmake instead of clang/CMakeLists.txt so that macro works in both clang and llvm. Commit a10f592: Port Classic Flang to LLVM 13 LLVM port from release_12x to release_13x, changes done in order to make project able to be build. Commit d385321 (partial): Change to Options.td in order to add the correct invocation for ffixed_line_length_VALUE. This commit also includes the following legacy patch: Commit 5da2c11: Add DebugInfo tests Commit c379c8d: [ClassicFlang][Driver] Correct the LLVM version passed by the Driver
Configuration menu - View commit details
-
Copy full SHA for 918f06a - Browse repository at this point
Copy the full SHA 918f06aView commit details -
[ClassicFlang][Driver] Coexist with LLVM Flang
Make TableGen respect the ENABLE_CLASSIC_FLANG macro and better separate the driver options that are mutually exclusive. When the macro is defined, do not build LLVM Flang at all.
Configuration menu - View commit details
-
Copy full SHA for 85181e4 - Browse repository at this point
Copy the full SHA 85181e4View commit details -
Support for DWARF 4/5 and fix of issues related to -gdwarf-X options (f…
…lang-compiler#92) * Support for -gdwarf-5 option in Flang driver. Summary: FLANG driver doesnt pass -gdwarf-4/5 to flang1 in form of xbits, while it passes for -gdwarf-2/3 -gdwarf-2 => -x 120 0x200 -gdwarf-3 => -x 120 0x4000 Due to this -gdwarf-5 is never honored and default option -gdwarf-4 is taken. # flang -gdwarf-5 test.f90 # llvm-dwarfdump a.out | grep version 0x00000000: Compile Unit: length = 0x0000008e version = 0x0004 Now 0x1000000/0x2000000 will be passed for -gdwarf-4/5 -gdwarf-4 => -x 120 0x1000000 -gdwarf-5 => -x 120 0x2000000 Testing: - GNU gdb fortran testsuite - check-llvm - check-debuginfo * Flang doenst choose correct dwarf version when multiple -g/-gdwarfN mentioned Summary: When multiple -g/-gdwarfN options are passed together at compile time, flang chooses the least one. Clang/gfortran etc choose the last one. -gdwarf-5 -gdwarf-3 => flang chooses 5 while clang/gfortran choose 3 -gdwarf-5 -g => flang choses the default while clang/gfortran choose 5 Testing: - check-llvm - check-debuginfo * Default dwarf version should be 4 for -g with flang Currently flang dumps dwarf version 2 for -g and 4 for absence of -g ------------------------- $ flang my.f90 $ llvm-dwarfdump a.out | grep version 0x00000000: Compile Unit: length = 0x0000003d version = 0x0004 abbr_offset = 0x0000 addr_size = 0x08 (next unit at 0x00000041) $ flang -g my.f90 $ llvm-dwarfdump a.out | grep version 0x00000000: Compile Unit: length = 0x00000047 version = 0x0002 abbr_offset = 0x0000 addr_size = 0x08 (next unit at 0x0000004b) ------------------------- It should be 4 for -g as it is the case with clang.
Configuration menu - View commit details
-
Copy full SHA for bde0e6a - Browse repository at this point
Copy the full SHA bde0e6aView commit details -
This commit is motivated by reducing the merge burden by shrinking the diff between llvm upstream and classic-flang-llvm-project. Outside of Flang, Fortran code is fed through the Compile phase, and the appropriate tooling is picked up through ToolChain::SelectTool. Classic Flang introduced a FortranFrontend, but these days this seems unnecessary. Fortran can go through the same machinery as everything else. * Use the Preprocess phase to preprocess Fortran code. This phase is always combined with the Compile phase. * Use the Compile phase to lower Fortran code to LLVM IR, and use the Backend phase to compile and optimize the IR. These phases are never combined. * Remove FortranFrontendJobClass. * Remove FortranFrontend tool (instead it's just the Flang tool, which in Classic Flang mode is Classic Flang). * Update tests which inspect the output of the Classic Flang tooling, and ensures that the driver does the right thing for various types of inputs. Based on a patch from Peter Waller <[email protected]>.
Configuration menu - View commit details
-
Copy full SHA for a4ebb0d - Browse repository at this point
Copy the full SHA a4ebb0dView commit details -
[workflows] Set up GitHub actions for Classic Flang
This commit squashes the following CI-related patches from the release_14x branch. commit 54a96d0 Author: Kiran Chandramohan <[email protected] Date: Wed Oct 28 17:08:22 2020 +0000 [workflows] Removing a few CI pipelines commit 8bf9551 Author: Bryan Chan <[email protected]> Date: Mon May 2 21:16:56 2022 -0400 [workflows] Stop using upstream actions and enable OpenMP build commit b921dd2 Author: michalpasztamobica <[email protected]> Date: Fri Jun 4 13:55:52 2021 +0200 [workflows] Allow pre-compile jobs to be trigerred manually. commit 9f828fd Author: michalpasztamobica <[email protected]> Date: Fri Jun 4 22:09:44 2021 +0200 [workflows] Add build scripts and make CI use them. commit a976b96 Author: Abraham Tovar <[email protected]> Date: Wed Oct 13 11:40:30 2021 +0200 [workflows] LLVM release_13x CI changes commit 7a5ee22 Author: Abraham Tovar <[email protected]> Date: Wed Nov 17 10:14:21 2021 +0000 [workflows] Removing CI test-base for release_100 commit a39a0c6 Author: pwisniewskimobica <[email protected]> Date: Thu Jan 27 15:06:19 2022 +0100 [workflows] Use GCC 11 in CI builds commit 198a6b2 Author: pwisniewskimobica <[email protected]> Date: Thu Jan 13 10:11:43 2022 +0100 [workflows] Modify CI scripts to add more test variants commit 84ed34d Author: Bryan Chan <[email protected]> Date: Sun Apr 24 19:13:25 2022 -0400 [workflows] support GitHub actions for release_14x branch commit dd852a6 Author: Bryan Chan <[email protected]> Date: Fri Jul 15 21:50:21 2022 -0400 [workflows] Fix CI for release_14x commit b52bcc0 Author: Bryan Chan <[email protected]> Date: Wed Jul 27 22:28:43 2022 -0400 [build] Stop caching AArch64 Docker image layers
Configuration menu - View commit details
-
Copy full SHA for e91f3a7 - Browse repository at this point
Copy the full SHA e91f3a7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8980f2a - Browse repository at this point
Copy the full SHA 8980f2aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 848e5a1 - Browse repository at this point
Copy the full SHA 848e5a1View commit details -
[DebugInfo]: Remove dangerous dead function (-Wreturn-stack-address)
Signed-off-by: Itay Bookstein <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 808ae62 - Browse repository at this point
Copy the full SHA 808ae62View commit details -
[ClassicFlang][Driver] Fix warnings
* -Wsuggest-override for ClassicFlangMacroBuilder::defineMacro * -Wrange-loop-construct when iterating over llvm::opt::Arg::getValues Signed-off-by: Itay Bookstein <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for d0a576f - Browse repository at this point
Copy the full SHA d0a576fView commit details -
[driver] Add -emit-flang-llvm option
-emit-flang-llvm instructs flang to stop after flang2 and dump the LLVM IR. Can be useful for debugging and also would be a useful option for testing flang output more accurately. Signed-off-by: Richard Barton <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 6f1a957 - Browse repository at this point
Copy the full SHA 6f1a957View commit details -
Configuration menu - View commit details
-
Copy full SHA for 56e39ad - Browse repository at this point
Copy the full SHA 56e39adView commit details -
[driver] Add infrastructure for testing the flang-driver in lit
Add %flang as a tool substitution available in lit tests. This apes the way %clang is defined and adds a $FLANG override in a similar vein. To avoid this being dead code, add a single test to check the flang driver is reporting the correct phases when running under various phase control options. Signed-off-by: Richard Barton <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 521bce1 - Browse repository at this point
Copy the full SHA 521bce1View commit details -
[LoopVectorize] Revert Classic Flang patch causing AArch64 test failure
This patch partially reverts the following commit from an early version of Classic Flang LLVM: flang-compiler/llvm@6866909 When the LoopVectorize.cpp change in that commit is ported to LLVM 14, it causes a failure in the following test: llvm/test/Transforms/LoopVectorize/AArch64/smallest-and-widest-types.ll Reverting the change allows all tests to run cleanly.
Configuration menu - View commit details
-
Copy full SHA for 8f79ace - Browse repository at this point
Copy the full SHA 8f79aceView commit details -
Configuration menu - View commit details
-
Copy full SHA for 12e5704 - Browse repository at this point
Copy the full SHA 12e5704View commit details -
[driver] Pass LLVM target_features to flang
Port commit 64bf2a6 to LLVM 16 and resolve merge conflicts. The original commit message follows: Pass LLVM target features (-mattr strings) to flang to embed in generated .ll files. For normal compilation this won't make much difference as the attributes are passed to clang after flang2 and can be applied there but this is crucial to enable LTO with flang as clang will not apply the attributes when building the flang2 output. libLTO will need to read these out of the object files to apply them. Signed-off-by: Richard Barton <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for ba329b6 - Browse repository at this point
Copy the full SHA ba329b6View commit details -
[flang-driver] Uniquify the target_features string
Use existing clang support for uniquifying target feature lists to pass a unique list through to flang.
Configuration menu - View commit details
-
Copy full SHA for b2afec8 - Browse repository at this point
Copy the full SHA b2afec8View commit details -
[driver] Pass msve-vector-bits to flang as vscale range
Signed-off-by: Paul Osmialowski <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for bffd285 - Browse repository at this point
Copy the full SHA bffd285View commit details -
Configuration menu - View commit details
-
Copy full SHA for 905be92 - Browse repository at this point
Copy the full SHA 905be92View commit details -
Fix classic flang version screen
Version screen has been erroneously reporting as flang-new since LLVM 12
Configuration menu - View commit details
-
Copy full SHA for d806652 - Browse repository at this point
Copy the full SHA d806652View commit details -
Canonicalize some ENABLE_CLASSIC_FLANG
These CMake variables are used to set python variables in the lit config. As such, they need to bet set with a valid CMake boolean value that is also a valid python boolean value (1, True, etc.) and not one that is not (ON, NO, etc.) to work. This is fragile. Call the LLVM cmake helper function on these two downstream CMake
Configuration menu - View commit details
-
Copy full SHA for f86adb9 - Browse repository at this point
Copy the full SHA f86adb9View commit details -
[NFC] Add missing ENABLE_CLASSIC_FLANG guard.
Signed-off-by: Richard Barton <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 0a2f946 - Browse repository at this point
Copy the full SHA 0a2f946View commit details -
Fix flang driver preprocessor issue
Fix a bug where flang -E -save-temps input.F90 would preprocess the input file twice. Add a new test for preprocessor behaviour that would expose the bug and updated fortran-phases test for new behaviour. Also added -emit-flang-llvm to that test for completeness. Signed-off-by: Richard Barton <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for c4014f0 - Browse repository at this point
Copy the full SHA c4014f0View commit details -
classic flang: enable IEEE by default (and sort Lower/Common inconsis…
…tency out) Signed-off-by: Paul Osmialowski <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for dbc4eae - Browse repository at this point
Copy the full SHA dbc4eaeView commit details -
Changes to flang's fp-contract behaviour
1) All fma and contraction behvaviour follows from the -ffp-contract= flag setting. 2) At optimization level 0, -ffp-contract=fast flag will not be honoured. 3) At all other levels, -ffp-contract=fast will be the default behaviour and the flag is honoured.
Configuration menu - View commit details
-
Copy full SHA for 884b849 - Browse repository at this point
Copy the full SHA 884b849View commit details -
[Driver] Remove the "-lompstub" in the driver
In some workloads, users get "-lompstub" by running "flang -v" and link it explicitly, which may cause failure when compiling programs with OpenMP (flang -fopenmp test.f90 -lompstub). There is no runtime call in ompstub.c generated when "-fopenmp" is not added to compile the program. When "-fopenmp" is added to compile the program, the runtime in llvm/openmp(libomp.so) will be used. If users use some runtime library routines such as "omp_get_thread_num" and compiles it without the option "-fopenmp", both gfortran and ifort report errors of "undefined reference to ...". After remove "-lompstub" in the driver, classic-flang reports the error, too. So, it's safe to remove the "-lompstub" in the driver.
Configuration menu - View commit details
-
Copy full SHA for 837a4f2 - Browse repository at this point
Copy the full SHA 837a4f2View commit details -
[Driver] Pass Classic Flang libraries to the linker correctly
In LLVM 15, the Fortran and OpenMP runtime libraries are added to the linker command line using common methods (addFortranRuntime* and addOpenMPRuntime*). This commit adds Classic Flang awareness to addFortranRuntime*, so that Classic Flang doesn't attempt to link with LLVM Flang libraries. Re-using the same methods as Clang and LLVM Flang also helps reduce downstream delta. A Classic Flang test is added to ensure that the linker command is constructed correctly.
Configuration menu - View commit details
-
Copy full SHA for 878551f - Browse repository at this point
Copy the full SHA 878551fView commit details -
[workflows] Allow GitHub actions in the classic-flang-llvm-project fork
LLVM 15 added 'ignore-forks: true' to the GitHub action triggers, which prevents the actions from running in the classic-flang-llvm-project repo. This patch undoes the setting.
Configuration menu - View commit details
-
Copy full SHA for d84b5bd - Browse repository at this point
Copy the full SHA d84b5bdView commit details -
[workflows] Update branch filter to allow actions on all 'release_*x'…
… branches By changing the branch filter from an explicit list to a glob, this commit hopes to reduce future maintenance burden (as long as we continue to use the same branch naming convention).
Configuration menu - View commit details
-
Copy full SHA for 2197917 - Browse repository at this point
Copy the full SHA 2197917View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2d2317a - Browse repository at this point
Copy the full SHA 2d2317aView commit details -
[workflows] Add AArch64 build job
Unlike their flang counterparts, classic-flang-llvm-project PRs did not have any AArch64 workflows. So problems with a PR may remain hidden until another PR is submitted to the flang repo and fails the AArch64 workflow there. This patch adds the missing build job.
Configuration menu - View commit details
-
Copy full SHA for 43c7876 - Browse repository at this point
Copy the full SHA 43c7876View commit details -
[AsmPrinter] Delete dead code; NFC
Delete some snippets that had been commented out since commit a10f592 in release_13x.
Configuration menu - View commit details
-
Copy full SHA for d7fc5b3 - Browse repository at this point
Copy the full SHA d7fc5b3View commit details -
[workflows] Delete misleading pull request template
Upstream llvm-project repository does not accept pull requests, but our fork does.
Configuration menu - View commit details
-
Copy full SHA for ce4532b - Browse repository at this point
Copy the full SHA ce4532bView commit details -
[workflows] Update path to Flang build directory
The following Flang commit has changed the default location of the Flang build directory, resulting in CI failures in the classic-flang-llvm-project: flang-compiler/flang@054bde3 This patch updates the path name in the CI workflows.
Configuration menu - View commit details
-
Copy full SHA for 6cc8126 - Browse repository at this point
Copy the full SHA 6cc8126View commit details -
[workflows] Add build script option to allow customization
This patch adds the -x option which lets the user specify additional CMake options when configuring classic-flang-llvm-project.
Configuration menu - View commit details
-
Copy full SHA for 13090ab - Browse repository at this point
Copy the full SHA 13090abView commit details -
Configuration menu - View commit details
-
Copy full SHA for cf0bd59 - Browse repository at this point
Copy the full SHA cf0bd59View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0db7fb8 - Browse repository at this point
Copy the full SHA 0db7fb8View commit details -
Changes for reassoc attributes
This patch adds code to generate the driver flags so that llvm bridge in flang2 can generate the nsz, reassoc attributes to arithmetic instructions. Also included is a testcase.
Configuration menu - View commit details
-
Copy full SHA for f4d4495 - Browse repository at this point
Copy the full SHA f4d4495View commit details -
[llvm] Provide separate veclib mapping of PGMATH functions for AArch64
This patch fixes link-time issues experienced while compiling some more obscure Fortran workloads. For those programs, calls to the libpgmath functions not defined for AArch64 were generated, e.g. `__pg_log_4` or `__fd_log_4`. This patch eliminates such possibility. Signed-off-by: Paul Osmialowski <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for c400474 - Browse repository at this point
Copy the full SHA c400474View commit details -
Delete libpgmath veclib definitions for sincos
As noted in flang-compiler#11, flang currently crashes when lowering a sincos reference into a libpgmath runtime function call. The issue is that `__fd_sincos_1` is defined as returning a `<{ double, double }>` struct and there is no LLVM support for automatically vectorizing target functions of this form. In particular, it is somewhat ambiguous how to vectorize such functions, i.e. how they pack their return values into the vector registers. libpgmath itself also has a somewhat questionable implementation of the vector forms of `sincos`, relying on this beauty: https://github.com/flang-compiler/flang/blob/master/runtime/libpgmath/lib/common/mth_vreturns.c#L8-L47 This may sometimes work in practice, but it is not particularly robust. For example, this will definitely break in any sort of LTO or instrumentation setting. I think until libpgmath is updated and LLVM upstream has a consensus on how to vectorize these functions, we just need to stop trying to vectorize these functions. As noted, since LLVM was crashing anyway, no performance and functionality is lost here over current master. Fixes flang-compiler#11. Originally By: Keno Fischer <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for daa8793 - Browse repository at this point
Copy the full SHA daa8793View commit details -
Configuration menu - View commit details
-
Copy full SHA for ffb0f81 - Browse repository at this point
Copy the full SHA ffb0f81View commit details -
win: Fix the build with verbose options
The 'append' method should be used instead of extend, since the 'extend' method treats the arguments as an iterable object, so extends the current list with list of characters.
Configuration menu - View commit details
-
Copy full SHA for 7c95495 - Browse repository at this point
Copy the full SHA 7c95495View commit details -
win: Don't install llvm by default
The 'llvm_build_project' build script installs llvm after the build by default. It is not necessary, if the user doesn't want.
Configuration menu - View commit details
-
Copy full SHA for 1bc9c18 - Browse repository at this point
Copy the full SHA 1bc9c18View commit details -
[Driver][ClassicFlang] Add options -fno-automatic and -f(no-)implicit…
…-none These three options are added in release_12. When compiling programs with them using Classic Flang, there are only warnings generated. They are removed now so that the errors of "unknown argument" are reported. Add these options in Driver for compatibility reason. Please note that these options are not implemented yet in Classic Flang.
Configuration menu - View commit details
-
Copy full SHA for f87c71c - Browse repository at this point
Copy the full SHA f87c71cView commit details -
[Driver] Don't use Classic Flang tools for LLVM IR input
LLVM IR input should not be acceptable for Classic Flang tools. This bug was caused by an incorrect merge conflict resolution when upgrading to LLVM 15.
Configuration menu - View commit details
-
Copy full SHA for 7fd89f5 - Browse repository at this point
Copy the full SHA 7fd89f5View commit details -
Setup Windows x64/arm64 GitHub action for Classic Flang LLVM
* Extend pre-compile_llvm workflow with a Windows job which defines build strategy for X64 and ARM64 architectures to produce prebuilt LLVM binaries for both platform. Currently Windows on ARM job is disabled, until an official will be available. * Also enable Classic Flang LLVM testing on Windows x64 only. Some tests are failing on Windows on ARM, so we don't run tests on WoA until they are fixed. * Added Windows support to 'classic-flang.f95' driver test. Modified FileChecker regex pattern to match Windows path separator and binary suffix too.
Configuration menu - View commit details
-
Copy full SHA for 4c1b125 - Browse repository at this point
Copy the full SHA 4c1b125View commit details -
script: update llvm build script
* Native Ninja is already available for Windows on ARM, so we can use it on both Windows platform. * Introduced a new '-i' build option to trigger whether the user wants to install LLVM or not. Previously the script relies on install_prefix option which was wrong approach. In this case we still could not defer the install for a later time if install directory was set. With this change the python build script will be consistent with official bash script.
Configuration menu - View commit details
-
Copy full SHA for 571d2b7 - Browse repository at this point
Copy the full SHA 571d2b7View commit details -
[Libomptarget] Add missing explicit moves on llvm::Error
Summary: Some older compilers, which we still support, have problems handling the copy elision that allows us to directly move an `Error` to an `Expected`. This patch adds explicit moves to remove the error. (cherry picked from commit edc0355)
Configuration menu - View commit details
-
Copy full SHA for 3817976 - Browse repository at this point
Copy the full SHA 3817976View commit details -
[LinkerWrapper] Fix
-Bsymbolic
being sent to the compiler and not t……he linker Summary: Clang doesn't warn on `-B` options passed to it. This one is not forwarded to the linker which results in some tests failing when offloading to x86_64 with the `bfd` linker. (cherry picked from f983987)
Configuration menu - View commit details
-
Copy full SHA for cb2d258 - Browse repository at this point
Copy the full SHA cb2d258View commit details -
[LinkerWrapper] Fix passing
-rpath
directly to clangSummary: This code passed the value of `-rpath` directly to the clang invocation. If we're using the linker then it'll be fine. However, if the linker is `gcc` as is the case when doing `-fopenmp-targets=x86_64` then this will cause problems. This patch adds the `-Wl,-rpath,` to feed it to the linker correctly. (cherry picked from 51ff548)
Configuration menu - View commit details
-
Copy full SHA for c86bb34 - Browse repository at this point
Copy the full SHA c86bb34View commit details -
[LinkerWrapper] Adjust placement of input files for the linker
Summary: The placement of input files can change the result of the linker. We should put the input files earlier to avoid this. (cherry picked from fa5209c)
Configuration menu - View commit details
-
Copy full SHA for 903e7e6 - Browse repository at this point
Copy the full SHA 903e7e6View commit details
Commits on Jul 5, 2023
-
[build] Specify LLVM_MAIN_SRC_DIR when running build-flang.sh
Otherwise Classic Flang's CMakeLists.txt will be unable to find llvm-lit and cannot generate test-related goals, e.g. `check-all`.
Configuration menu - View commit details
-
Copy full SHA for c6e8575 - Browse repository at this point
Copy the full SHA c6e8575View commit details