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
