dankm wrote:
If people are good with this, I don't have commit access, so I'd appreciate
somebody merging.
https://github.com/llvm/llvm-project/pull/141250
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/
@@ -1352,7 +1352,12 @@ static StringRef
getIdentStringFromSourceLocation(CodeGenFunction &CGF,
llvm::raw_svector_ostream OS(Buffer);
// Build debug location
PresumedLoc PLoc = CGF.getContext().getSourceManager().getPresumedLoc(Loc);
- OS << ";" << PLoc.getFilename() <<
dankm wrote:
> @jdoerfert it looks like you wrote the original code I'm modifying here, so
> I'd like your feedback. Don't know who else would be interested here.
Hm, @alexey-bataev looks like this is arguably conformance related too.
https://github.com/llvm/llvm-project/pull/141250
__
https://github.com/dankm updated
https://github.com/llvm/llvm-project/pull/141250
>From e4fc23b1486180ec01a68f0d2c367d8d15aa8203 Mon Sep 17 00:00:00 2001
From: Dan McGregor
Date: Fri, 23 May 2025 10:19:22 -0600
Subject: [PATCH] [Clang] Remap paths in OpenMP runtime calls
Apply the debug prefix
dankm wrote:
Neat, the original was good, and apparently some tests don't run on my system.
Going back to to that version :/
https://github.com/llvm/llvm-project/pull/141250
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.or
https://github.com/dankm updated
https://github.com/llvm/llvm-project/pull/141250
>From 8e534d8fe14d70e4fa65dc6d8dad8ad4de73caae Mon Sep 17 00:00:00 2001
From: Dan McGregor
Date: Fri, 23 May 2025 10:19:22 -0600
Subject: [PATCH] [Clang] Remap paths in OpenMP runtime calls
Apply the debug prefix
@@ -1370,10 +1375,14 @@ llvm::Value
*CGOpenMPRuntime::emitUpdateLocation(CodeGenFunction &CGF,
SrcLocStr = OMPBuilder.getOrCreateDefaultSrcLocStr(SrcLocStrSize);
} else {
std::string FunctionName;
+std::string FileName;
if (const auto *FD = dyn_cast_or_null(
dankm wrote:
@jdoerfert it looks like you wrote the original code I'm modifying here, so I'd
like your feedback. Don't know who else would be interested here.
https://github.com/llvm/llvm-project/pull/141250
___
cfe-commits mailing list
cfe-commits@li
@@ -1370,10 +1375,14 @@ llvm::Value
*CGOpenMPRuntime::emitUpdateLocation(CodeGenFunction &CGF,
SrcLocStr = OMPBuilder.getOrCreateDefaultSrcLocStr(SrcLocStrSize);
} else {
std::string FunctionName;
+std::string FileName;
if (const auto *FD = dyn_cast_or_null(
https://github.com/dankm updated
https://github.com/llvm/llvm-project/pull/141250
>From 1478c326a7e60b270f2a81e373c390183d1483ed Mon Sep 17 00:00:00 2001
From: Dan McGregor
Date: Fri, 23 May 2025 10:19:22 -0600
Subject: [PATCH] [Clang] Remap paths in OpenMP runtime calls
Apply the debug prefix
https://github.com/dankm updated
https://github.com/llvm/llvm-project/pull/141250
>From 7fdc69f5c55074fa9d647b01f4a370a34c26f966 Mon Sep 17 00:00:00 2001
From: Dan McGregor
Date: Fri, 23 May 2025 10:19:22 -0600
Subject: [PATCH] [Clang] Remap paths in OpenMP runtime calls
Apply the debug prefix
dankm wrote:
I do need to add tests for this, but wanted to get the general idea up for
comments.
https://github.com/llvm/llvm-project/pull/141250
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/
https://github.com/dankm created
https://github.com/llvm/llvm-project/pull/141250
Apply the debug prefix mapping to the OpenMP location strings.
Fixes https://github.com/llvm/llvm-project/issues/82541
>From 6a6c63abc2ff2dd594f82a5ad92f01aee5bdd37a Mon Sep 17 00:00:00 2001
From: Dan McGregor
D
https://github.com/dankm updated https://github.com/llvm/llvm-project/pull/77675
>From 588e279c3a1c4dcbea9c2d262848eb1d7d49d0d2 Mon Sep 17 00:00:00 2001
From: Dan McGregor
Date: Wed, 10 Jan 2024 13:33:37 -0600
Subject: [PATCH] [flang] include sys/wait.h for EXECUTE_COMMAND_LINE
Linux defines WE
@@ -0,0 +1,206 @@
+//===-- runtime/execute.cpp
---===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Ap
@@ -0,0 +1,206 @@
+//===-- runtime/execute.cpp
---===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Ap
https://github.com/dankm edited https://github.com/llvm/llvm-project/pull/65812
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/dankm edited https://github.com/llvm/llvm-project/pull/65812
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
dankm wrote:
Sounds good. Everything I want is in the pull request, I'm going to create a
new pull request to update some tests as we discussed above. I'll update the
description to be a bit more clear.
Thanks for all help.
https://github.com/llvm/llvm-project/pull/65812
_
dankm wrote:
Hm. I have "fixup" commits in this branch, should I rebase those, or can we
squash merge as-is?
https://github.com/llvm/llvm-project/pull/65812
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman
dankm wrote:
No _good_ particular reason. I got hung up on the formatting issues then ran
out of steam, and busy with $job. I just ran clang-format on this change and it
came up clean.
And now that I've done that the only reason I have left is I'm unable to merge
my own changes. Would you min
21 matches
Mail list logo
