@@ -40,6 +41,7 @@ using namespace llvm;
namespace lld::wasm {
static std::unique_ptr createLTO() {
lto::Config c;
+ c.EmbedCmdArgs = commonContext().cmdArgs;
sbc100 wrote:
I think it might be the other way around. It looks like only COFF currently
support
https://github.com/mrexodia edited
https://github.com/llvm/llvm-project/pull/79390
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -40,6 +41,7 @@ using namespace llvm;
namespace lld::wasm {
static std::unique_ptr createLTO() {
lto::Config c;
+ c.EmbedCmdArgs = commonContext().cmdArgs;
mrexodia wrote:
Thanks for the comment! I don't think it makes a difference, except that
`context(
@@ -40,6 +41,7 @@ using namespace llvm;
namespace lld::wasm {
static std::unique_ptr createLTO() {
lto::Config c;
+ c.EmbedCmdArgs = commonContext().cmdArgs;
sbc100 wrote:
It looks like `commonContext()` is not actually used elsewhere in lld, and this
shou
https://github.com/mrexodia updated
https://github.com/llvm/llvm-project/pull/79390
>From d60c2ad5bc88d6b64d7b3d6b179e303e3eeb936c Mon Sep 17 00:00:00 2001
From: Duncan Ogilvie
Date: Thu, 25 Jan 2024 00:08:49 +0100
Subject: [PATCH] Embed the command line arguments during LTO
---
clang/lib/Cod
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff 2e81ac25b4e2bfdc71aac19a911525a7f35680be
5a7c330982873c9198ef8c6f90d10f3ccb9ac5d9 --
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
@llvm/pr-subscribers-lld
Author: Duncan Ogilvie (mrexodia)
Changes
This was previously marked as `FIXME` in `LTOBackend.cpp`.
List of changes:
- Removed the unused `lto::Config.MllvmArgs` field (this field was only
assigned, never actuall
https://github.com/mrexodia edited
https://github.com/llvm/llvm-project/pull/79390
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
github-actions[bot] wrote:
Thank you for submitting a Pull Request (PR) to the LLVM Project!
This PR will be automatically labeled and the relevant teams will be
notified.
If you wish to, you can add reviewers by using the "Reviewers" section on this
page.
If this is not working for you, it i
https://github.com/mrexodia created
https://github.com/llvm/llvm-project/pull/79390
This was previously marked as `FIXME` in `LTOBackend.cpp`.
List of changes:
- Removed the unused `lto::Config.MllvmArgs` field
- Add a new `lto::Config.EmbedCmdArgs` field (`\0`-separated list, since that's
how
10 matches
Mail list logo
