Author: kparzysz
Date: Fri May 11 09:41:51 2018
New Revision: 332105
URL: http://llvm.org/viewvc/llvm-project?rev=332105&view=rev
Log:
[Hexagon] Implement checking arguments of builtin calls
Added:
cfe/trunk/test/CodeGen/hexagon-check-builtins.c
Modified:
cfe/trunk/include/clang/Sema/Sema
Author: kparzysz
Date: Tue May 15 11:15:59 2018
New Revision: 332383
URL: http://llvm.org/viewvc/llvm-project?rev=332383&view=rev
Log:
[Hexagon] Add driver options for subtarget features
Added:
cfe/trunk/test/Driver/hexagon-memops.c
cfe/trunk/test/Driver/hexagon-nvj.c
cfe/trunk/test/D
Author: kparzysz
Date: Fri Aug 24 10:13:42 2018
New Revision: 340622
URL: http://llvm.org/viewvc/llvm-project?rev=340622&view=rev
Log:
[Hexagon] Remove unneeded strings from builtin definitions, NFC
Modified:
cfe/trunk/include/clang/Basic/BuiltinsHexagon.def
Modified: cfe/trunk/include/clang
Author: kparzysz
Date: Thu May 25 07:55:47 2017
New Revision: 303851
URL: http://llvm.org/viewvc/llvm-project?rev=303851&view=rev
Log:
[CodeGen] Pessimize aliasing for member unions (and may-alias) objects
Use the TBAA info of the omnipotent char for these objects.
Differential Revision: https:/
Author: kparzysz
Date: Wed Feb 28 12:31:55 2018
New Revision: 326366
URL: http://llvm.org/viewvc/llvm-project?rev=326366&view=rev
Log:
[Hexagon] Add -ffixed-r19 driver option and translate it to +reserved-r19
Modified:
cfe/trunk/include/clang/Driver/Options.td
cfe/trunk/lib/Driver/ToolCha
Author: kparzysz
Date: Tue Mar 13 06:30:43 2018
New Revision: 327393
URL: http://llvm.org/viewvc/llvm-project?rev=327393&view=rev
Log:
[Hexagon] Clang side of r327302 in LLVM
Add option -m[no-]packets to control generation of instruction packets
(enabled by default).
Added:
cfe/trunk/test/Dr
Author: kparzysz
Date: Fri Oct 19 08:36:45 2018
New Revision: 344786
URL: http://llvm.org/viewvc/llvm-project?rev=344786&view=rev
Log:
[Hexagon] Remove support for V4
Modified:
cfe/trunk/docs/ClangCommandLineReference.rst
cfe/trunk/include/clang/Driver/Options.td
cfe/trunk/lib/Basic/T
Author: kparzysz
Date: Wed Oct 24 10:55:18 2018
New Revision: 345170
URL: http://llvm.org/viewvc/llvm-project?rev=345170&view=rev
Log:
[Hexagon] Flip hexagon-autohvx to be true by default
This will allow other generators of LLVM IR to use the auto-vectorizer
without having to change that flag.
N
Author: kparzysz
Date: Mon Dec 3 15:52:33 2018
New Revision: 348214
URL: http://llvm.org/viewvc/llvm-project?rev=348214&view=rev
Log:
[Hexagon] Fix intrinsic test
Modified:
cfe/trunk/test/CodeGen/builtins-hexagon.c
Modified: cfe/trunk/test/CodeGen/builtins-hexagon.c
URL:
http://llvm.org/vi
Author: kparzysz
Date: Wed Dec 5 13:38:35 2018
New Revision: 348415
URL: http://llvm.org/viewvc/llvm-project?rev=348415&view=rev
Log:
[Hexagon] Add support for Hexagon V66
Modified:
cfe/trunk/include/clang/Driver/Options.td
cfe/trunk/lib/Basic/Targets/Hexagon.cpp
cfe/trunk/lib/Driver
This should to to cfe-commits. Redirecting.
-Krzysztof
On 9/8/2017 10:25 AM, Antoni Boucher via llvm-commits wrote:
Hello.
I've fixed the bug 27628:
https://bugs.llvm.org/show_bug.cgi?id=27628
I attached the patch.
Thanks.
___
llvm-commits mailin
Aaand the patch itself...
-K
On 9/8/2017 10:32 AM, Krzysztof Parzyszek via cfe-commits wrote:
This should to to cfe-commits. Redirecting.
-Krzysztof
On 9/8/2017 10:25 AM, Antoni Boucher via llvm-commits wrote:
Hello.
I've fixed the bug 27628:
https://bugs.llvm.org/show_bug.cgi?id=276
Author: kparzysz
Date: Fri Jun 28 08:08:03 2019
New Revision: 364648
URL: http://llvm.org/viewvc/llvm-project?rev=364648&view=rev
Log:
[Hexagon] driver uses out-of-date option name and binary name
Patch by A. Skrobov (t.yomitch).
Differential Revision: https://reviews.llvm.org/D62127
Modified:
Author: kparzysz
Date: Thu Apr 12 09:25:35 2018
New Revision: 329923
URL: http://llvm.org/viewvc/llvm-project?rev=329923&view=rev
Log:
[Hexagon] Enable auto-vectorization only when -fvectorize was given
Added:
cfe/trunk/test/Driver/hexagon-vectorize.c
Modified:
cfe/trunk/lib/Driver/ToolCh
Author: kparzysz
Date: Mon Apr 16 12:11:17 2018
New Revision: 330150
URL: http://llvm.org/viewvc/llvm-project?rev=330150&view=rev
Log:
[Hexagon] Emit a warning when -fvectorize is given without -mhvx
Modified:
cfe/trunk/include/clang/Basic/DiagnosticDriverKinds.td
cfe/trunk/lib/Driver/Too
Author: kparzysz
Date: Thu Jul 12 11:54:04 2018
New Revision: 336933
URL: http://llvm.org/viewvc/llvm-project?rev=336933&view=rev
Log:
[Hexagon] Diagnose intrinsics not supported by selected CPU/HVX
Added:
cfe/trunk/test/Sema/builtins-hexagon-v55.c
cfe/trunk/test/Sema/builtins-hexagon-v60
Author: kparzysz
Date: Fri Jul 13 14:32:33 2018
New Revision: 337049
URL: http://llvm.org/viewvc/llvm-project?rev=337049&view=rev
Log:
[Hexagon] Fix hvx-length feature name in testcases
Modified:
cfe/trunk/test/CodeGen/builtins-hvx128.c
cfe/trunk/test/CodeGen/builtins-hvx64.c
Modified: c
Author: Krzysztof Parzyszek
Date: 2019-10-29T08:56:01-05:00
New Revision: 99f51960fdb5559d6720281bff9a63041452bf9a
URL:
https://github.com/llvm/llvm-project/commit/99f51960fdb5559d6720281bff9a63041452bf9a
DIFF:
https://github.com/llvm/llvm-project/commit/99f51960fdb5559d6720281bff9a63041452bf9a
Author: Krzysztof Parzyszek
Date: 2020-08-10T10:37:24-05:00
New Revision: 7406eb4f6afd8df9bd4dbb918f5e7005ba71d58c
URL:
https://github.com/llvm/llvm-project/commit/7406eb4f6afd8df9bd4dbb918f5e7005ba71d58c
DIFF:
https://github.com/llvm/llvm-project/commit/7406eb4f6afd8df9bd4dbb918f5e7005ba71d58c
kparzysz added a comment.
dbgs showed that the path components were `.. target hexagon include`.
The paths are constructed in lib/Driver/ToolChains.cpp, many are based on
"getHexagonTargetDir".
https://reviews.llvm.org/D25597
___
cfe-commits mailin
kparzysz added a comment.
Right on entry to the asserting function:
(gdb) where
#0 0x752b9870 in (anonymous
namespace)::RedirectingFileSystem::lookupPath(llvm::sys::path::const_iterator,
llvm::sys::path::const_iterator, (anonymous namespace)::Entry*) () from
/w/bld/up/bin/../lib/l
kparzysz added a comment.
The `..:target:hexagon:include:` is debug code printing all the path
components, separated by :.
https://reviews.llvm.org/D25597
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman
kparzysz added a comment.
This change
bool FileManager::getStatValue(StringRef Path, FileData &Data, bool isFile,
std::unique_ptr *F) {
// FIXME: FileSystemOpts shouldn't be passed in here, all paths should be
// absolute!
llvm::dbgs() << "FileSystemOp
kparzysz added a comment.
In https://reviews.llvm.org/D25597#571532, @kparzysz wrote:
> `FileSystemOpts.WorkingDir: ''`
There is no space between the single quotes.
https://reviews.llvm.org/D25597
___
cfe-commits mailing list
cfe-commits@lists.ll
kparzysz added a comment.
Printing Path shows
`/../target/hexagon/include`
https://reviews.llvm.org/D25597
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: kparzysz
Date: Mon Oct 17 08:23:41 2016
New Revision: 284383
URL: http://llvm.org/viewvc/llvm-project?rev=284383&view=rev
Log:
Return correct path from HexagonToolChain::getHexagonTargetDir
This problem was exposed by r284129, causing clang-hexagon-elf to fail
clang tests.
Modified:
Hi,
This broke build on FreeBSD 11:
[100%] Building CXX object
tools/clang/tools/extra/include-fixer/find-all-symbols/CMakeFiles/findAllSymbols.dir/HeaderMapCollector.cpp.o
cd /w/bld/org/tools/clang/tools/extra/include-fixer/find-all-symbols &&
/w/c/clang+llvm-5.0.0-x86_64-unknown-freebsd11/bin
There has been some problem with std::pair on FreeBSD (due to some ABI
compatibility issue), but I don't know much about what it was.
Commenting the reserve doesn't help, unfortunately. The same problem is
now flagged in emplace_back.
-Krzysztof
On 11/23/2017 11:47 AM, Benjamin Kramer wrote
+Dimitry.
On 11/23/2017 12:50 PM, Benjamin Kramer wrote:
I'm afraid I can't really help you here. You can try twiddling the
code a bit by creating the pair explicitly and moving it into the
vector, just to avoid the brokenness in the standard library. Not sure
if that will help though.
On Thu,
After upgrading my FreeBSD to the latest -STABLE this no longer fails.
-Krzysztof
PS. And system clang was upgraded to 5.0.0. Nice!
On 11/25/2017 11:20 AM, Dimitry Andric wrote:
Yeah, in the past libc++ broke its own ABI with
http://llvm.org/viewvc/llvm-project?view=revision&revision=194536,
Author: kparzysz
Date: Mon Dec 11 11:29:56 2017
New Revision: 320410
URL: http://llvm.org/viewvc/llvm-project?rev=320410&view=rev
Log:
[Hexagon] Remove unsupported vlut intrinsics
Modified:
cfe/trunk/test/CodeGen/builtins-hexagon.c
Modified: cfe/trunk/test/CodeGen/builtins-hexagon.c
URL:
ht
Author: kparzysz
Date: Wed Dec 13 05:48:07 2017
New Revision: 320579
URL: http://llvm.org/viewvc/llvm-project?rev=320579&view=rev
Log:
[Hexagon] Add front-end support for Hexagon V65
Modified:
cfe/trunk/include/clang/Driver/Options.td
cfe/trunk/lib/Basic/Targets/Hexagon.cpp
cfe/trunk/
kparzysz wrote:
> The address spaces for AMDGPU defined
> [here](https://github.com/llvm/llvm-project/blob/main/llvm/lib/Target/AMDGPU/AMDGPU.h#L395-L456)
> contain more types of address spaces in comparison to the enum defined in
> clang. Is it ok to extend number of address space types for c
kparzysz wrote:
> I'd just move these definitions to their own file, and use that file in
> clang/flang/llvm backend. So, indirectly, yes.
Sorry, didn't notice you have already created a separate file: 👍 I'd just use
the address space identifiers that you quoted instead.
https://github.com/ll
@@ -0,0 +1,31 @@
+//=== AMDGPUAddrSpace.h -*- C++
-*-===//
+//
+// 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: Apa
https://github.com/kparzysz approved this pull request.
LGTM, thanks!
https://github.com/llvm/llvm-project/pull/73944
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kparzysz created
https://github.com/llvm/llvm-project/pull/72664
```
clang/lib/Tooling/Inclusions/Stdlib/StandardLibrary.cpp:144:65: warning:
narrowing conversion of ‘llvm::StringRef(((const
char*)"std::experimental::filesystem::")).llvm::StringRef::size()’ from
‘size_t’ {a
https://github.com/kparzysz edited
https://github.com/llvm/llvm-project/pull/72664
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kparzysz updated
https://github.com/llvm/llvm-project/pull/72664
>From a6c711d61221558bad907a8e1a4944260d38c6ae Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek
Date: Fri, 17 Nov 2023 09:18:02 -0600
Subject: [PATCH] [Tooling/Inclusion] Avoid narrowing conversions in macro
https://github.com/kparzysz closed
https://github.com/llvm/llvm-project/pull/72664
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Krzysztof Parzyszek
Date: 2023-11-20T10:48:06-06:00
New Revision: ddfed815c930979414d403e01caca23875072676
URL:
https://github.com/llvm/llvm-project/commit/ddfed815c930979414d403e01caca23875072676
DIFF:
https://github.com/llvm/llvm-project/commit/ddfed815c930979414d403e01caca23875072676
https://github.com/kparzysz updated
https://github.com/llvm/llvm-project/pull/75523
>From 21261fef67dbbea956adf2e09b8abacd92d9caf3 Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek
Date: Thu, 14 Dec 2023 13:45:17 -0600
Subject: [PATCH 1/3] [flang][OpenMP] Move handling of OpenMP symbol flags t
https://github.com/kparzysz closed
https://github.com/llvm/llvm-project/pull/75523
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -977,14 +977,51 @@ bool tools::addOpenMPRuntime(ArgStringList &CmdArgs,
const ToolChain &TC,
return true;
}
-void tools::addFortranRuntimeLibs(const ToolChain &TC,
+void tools::addFortranRuntimeLibs(const ToolChain &TC, const ArgList &Args,
@@ -977,14 +977,51 @@ bool tools::addOpenMPRuntime(ArgStringList &CmdArgs,
const ToolChain &TC,
return true;
}
-void tools::addFortranRuntimeLibs(const ToolChain &TC,
+void tools::addFortranRuntimeLibs(const ToolChain &TC, const ArgList &Args,
@@ -977,14 +977,51 @@ bool tools::addOpenMPRuntime(ArgStringList &CmdArgs,
const ToolChain &TC,
return true;
}
-void tools::addFortranRuntimeLibs(const ToolChain &TC,
+void tools::addFortranRuntimeLibs(const ToolChain &TC, const ArgList &Args,
https://github.com/kparzysz approved this pull request.
https://github.com/llvm/llvm-project/pull/73124
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -977,14 +977,63 @@ bool tools::addOpenMPRuntime(ArgStringList &CmdArgs,
const ToolChain &TC,
return true;
}
-void tools::addFortranRuntimeLibs(const ToolChain &TC,
+void tools::addFortranRuntimeLibs(const ToolChain &TC, const ArgList &Args,
https://github.com/kparzysz edited
https://github.com/llvm/llvm-project/pull/73124
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -977,14 +977,61 @@ bool tools::addOpenMPRuntime(ArgStringList &CmdArgs,
const ToolChain &TC,
return true;
}
-void tools::addFortranRuntimeLibs(const ToolChain &TC,
+void tools::addFortranRuntimeLibs(const ToolChain &TC, const ArgList &Args,
@@ -977,14 +977,61 @@ bool tools::addOpenMPRuntime(ArgStringList &CmdArgs,
const ToolChain &TC,
return true;
}
-void tools::addFortranRuntimeLibs(const ToolChain &TC,
+void tools::addFortranRuntimeLibs(const ToolChain &TC, const ArgList &Args,
@@ -977,14 +977,61 @@ bool tools::addOpenMPRuntime(ArgStringList &CmdArgs,
const ToolChain &TC,
return true;
}
-void tools::addFortranRuntimeLibs(const ToolChain &TC,
+void tools::addFortranRuntimeLibs(const ToolChain &TC, const ArgList &Args,
https://github.com/kparzysz approved this pull request.
https://github.com/llvm/llvm-project/pull/73124
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kparzysz closed
https://github.com/llvm/llvm-project/pull/73124
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -31,6 +31,15 @@ class Triple;
// back-end to TableGen to create these clean tables.
namespace AMDGPU {
+/// Address space values for AMD GPUs
+enum AddrSpace {
+ Generic = 0,
kparzysz wrote:
We could change this enum to `enum class`.
https://github.com/l
kparzysz wrote:
> I think the enum looks a bit out of place in a file called "TargetParser.h",
> but I'm not very familiar with this part of the project, so I'll let the
> experts say something if there is a more suitable location for it.
That's a good point. The address spaces for AMDGPU are
Author: Krzysztof Parzyszek
Date: 2022-02-08T09:06:15-08:00
New Revision: 2ecda9ec9cc89b87ce2e56452229c9444beabf21
URL:
https://github.com/llvm/llvm-project/commit/2ecda9ec9cc89b87ce2e56452229c9444beabf21
DIFF:
https://github.com/llvm/llvm-project/commit/2ecda9ec9cc89b87ce2e56452229c9444beabf21
Author: Krzysztof Parzyszek
Date: 2022-05-09T07:11:56-07:00
New Revision: d9e6b5df74f54a7cf21a419f737d922040c1ed08
URL:
https://github.com/llvm/llvm-project/commit/d9e6b5df74f54a7cf21a419f737d922040c1ed08
DIFF:
https://github.com/llvm/llvm-project/commit/d9e6b5df74f54a7cf21a419f737d922040c1ed08
https://github.com/kparzysz created
https://github.com/llvm/llvm-project/pull/85217
…ctive
The function `ActOnOpenMPTargetParallelForSimdDirective` gets the number of
capture levels for OMPD_target_parallel_for, whereas the intended directive is
OMPD_target_parallel_for_simd.
>From 717561a9d
kparzysz wrote:
I suspect this was a mistake, let me know if you intended to use the existing
directive here.
https://github.com/llvm/llvm-project/pull/85217
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailma
Author: Krzysztof Parzyszek
Date: 2024-03-25T16:51:11-05:00
New Revision: cb994d41c3afb2bd0b25a4c5b2ac48978bf1b23d
URL:
https://github.com/llvm/llvm-project/commit/cb994d41c3afb2bd0b25a4c5b2ac48978bf1b23d
DIFF:
https://github.com/llvm/llvm-project/commit/cb994d41c3afb2bd0b25a4c5b2ac48978bf1b23d
https://github.com/kparzysz created
https://github.com/llvm/llvm-project/pull/8
NFC
>From 3ffabe11eb01f42ceaeb92895a0eb0f251e6ef9a Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek
Date: Thu, 21 Mar 2024 06:54:02 -0500
Subject: [PATCH] [clang][OpenMP] Add const-qualified `getIteratorModif
kparzysz wrote:
This brings a question: most of such accessors return `Expr *` in both const
and non-const versions. If that's intended, does it make sense to even have
non-const variants of these accessors?
https://github.com/llvm/llvm-project/pull/8
_
kparzysz wrote:
I don't think so, because nothing is failing without it. This is just for
completeness.
https://github.com/llvm/llvm-project/pull/8
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/lis
kparzysz wrote:
> Are you going to submit your code?
Yes, but it's still incomplete. I can wait with this PR if you prefer.
https://github.com/llvm/llvm-project/pull/8
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org
https://github.com/kparzysz closed
https://github.com/llvm/llvm-project/pull/8
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kparzysz wrote:
Will revisit this when the using code is ready for review/merge.
https://github.com/llvm/llvm-project/pull/8
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -342,4 +359,22 @@ def TDL_DirA : Directive<"dira"> {
// IMPL-NEXT:llvm_unreachable("Invalid Tdl Directive kind");
// IMPL-NEXT: }
// IMPL-EMPTY:
+// IMPL-NEXT: llvm::ArrayRef
llvm::tdl::getLeafConstructs(llvm::tdl::Directive Dir) {
+// IMPL-NEXT:static llvm::Array
https://github.com/kparzysz edited
https://github.com/llvm/llvm-project/pull/83625
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kparzysz updated
https://github.com/llvm/llvm-project/pull/83625
>From b62919c2ce24feb3c75a5bbecce3d6b6ee8e5b7e Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek
Date: Tue, 16 Jan 2024 16:40:47 -0600
Subject: [PATCH 1/9] [Frontend] Add leaf constructs and association to
Ope
@@ -342,4 +359,22 @@ def TDL_DirA : Directive<"dira"> {
// IMPL-NEXT:llvm_unreachable("Invalid Tdl Directive kind");
// IMPL-NEXT: }
// IMPL-EMPTY:
+// IMPL-NEXT: llvm::ArrayRef
llvm::tdl::getLeafConstructs(llvm::tdl::Directive Dir) {
+// IMPL-NEXT:static llvm::Array
https://github.com/kparzysz edited
https://github.com/llvm/llvm-project/pull/83625
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kparzysz edited
https://github.com/llvm/llvm-project/pull/83625
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -435,6 +450,217 @@ static void GenerateIsAllowedClause(const
DirectiveLanguage &DirLang,
OS << "}\n"; // End of function isAllowedClauseForDirective
}
+// Generate the getLeafConstructs function implementation.
+static void GenerateGetLeafConstructs(const DirectiveLangua
kparzysz wrote:
Hey @erichkeane, do you have anything to add? I have replaced `leafs` in the
.td file with `leafConstructs`, and with `leaves` elsewhere.
https://github.com/llvm/llvm-project/pull/83625
___
cfe-commits mailing list
cfe-commits@lists.l
https://github.com/kparzysz closed
https://github.com/llvm/llvm-project/pull/83625
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kparzysz created
https://github.com/llvm/llvm-project/pull/84817
This doesn't do anything with clauses yet. It's just hooking things up to
handle leaf constructs, and it's not even trying to cover all cases.
>From 45859d3b1fb914d26831086fb7f7e6decdec9010 Mon Sep 17 00:00:00
https://github.com/kparzysz created
https://github.com/llvm/llvm-project/pull/83625
Add members "leafs" and "association" to .td describing OpenMP/ACC directives:
"leafs" are the leaf constructs for composite/combined constructs, and
"association" is the source language construct to which the
https://github.com/kparzysz updated
https://github.com/llvm/llvm-project/pull/83625
>From b62919c2ce24feb3c75a5bbecce3d6b6ee8e5b7e Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek
Date: Tue, 16 Jan 2024 16:40:47 -0600
Subject: [PATCH] [Frontend] Add leaf constructs and association to OpenMP/A
https://github.com/kparzysz updated
https://github.com/llvm/llvm-project/pull/83625
>From b62919c2ce24feb3c75a5bbecce3d6b6ee8e5b7e Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek
Date: Tue, 16 Jan 2024 16:40:47 -0600
Subject: [PATCH 1/2] [Frontend] Add leaf constructs and association to
Ope
@@ -665,60 +619,44 @@ bool
clang::isOpenMPTargetDataManagementDirective(OpenMPDirectiveKind DKind) {
}
bool clang::isOpenMPNestingTeamsDirective(OpenMPDirectiveKind DKind) {
- return DKind == OMPD_teams || DKind == OMPD_teams_distribute ||
- DKind == OMPD_teams_distr
@@ -665,60 +619,44 @@ bool
clang::isOpenMPTargetDataManagementDirective(OpenMPDirectiveKind DKind) {
}
bool clang::isOpenMPNestingTeamsDirective(OpenMPDirectiveKind DKind) {
- return DKind == OMPD_teams || DKind == OMPD_teams_distribute ||
- DKind == OMPD_teams_distr
@@ -665,60 +619,44 @@ bool
clang::isOpenMPTargetDataManagementDirective(OpenMPDirectiveKind DKind) {
}
bool clang::isOpenMPNestingTeamsDirective(OpenMPDirectiveKind DKind) {
- return DKind == OMPD_teams || DKind == OMPD_teams_distribute ||
- DKind == OMPD_teams_distr
kparzysz wrote:
I removed the check for `end do simd` with a comment that it can't appear in
clang. I think you deleted that remark, let me know if I should bring the
check back.
https://github.com/llvm/llvm-project/pull/83625
___
cfe-commits mailin
https://github.com/kparzysz updated
https://github.com/llvm/llvm-project/pull/83625
>From b62919c2ce24feb3c75a5bbecce3d6b6ee8e5b7e Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek
Date: Tue, 16 Jan 2024 16:40:47 -0600
Subject: [PATCH 1/3] [Frontend] Add leaf constructs and association to
Ope
@@ -665,60 +619,44 @@ bool
clang::isOpenMPTargetDataManagementDirective(OpenMPDirectiveKind DKind) {
}
bool clang::isOpenMPNestingTeamsDirective(OpenMPDirectiveKind DKind) {
- return DKind == OMPD_teams || DKind == OMPD_teams_distribute ||
- DKind == OMPD_teams_distr
@@ -665,60 +619,44 @@ bool
clang::isOpenMPTargetDataManagementDirective(OpenMPDirectiveKind DKind) {
}
bool clang::isOpenMPNestingTeamsDirective(OpenMPDirectiveKind DKind) {
- return DKind == OMPD_teams || DKind == OMPD_teams_distribute ||
- DKind == OMPD_teams_distr
https://github.com/kparzysz updated
https://github.com/llvm/llvm-project/pull/83625
>From b62919c2ce24feb3c75a5bbecce3d6b6ee8e5b7e Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek
Date: Tue, 16 Jan 2024 16:40:47 -0600
Subject: [PATCH 1/3] [Frontend] Add leaf constructs and association to
Ope
kparzysz wrote:
The failing test on Windows is unrelated. I've seen it fail in other builds
before.
https://github.com/llvm/llvm-project/pull/83625
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinf
https://github.com/kparzysz updated
https://github.com/llvm/llvm-project/pull/83625
>From b62919c2ce24feb3c75a5bbecce3d6b6ee8e5b7e Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek
Date: Tue, 16 Jan 2024 16:40:47 -0600
Subject: [PATCH 1/3] [Frontend] Add leaf constructs and association to
Ope
@@ -619,44 +595,22 @@ bool
clang::isOpenMPWorksharingDirective(OpenMPDirectiveKind DKind) {
}
bool clang::isOpenMPTaskLoopDirective(OpenMPDirectiveKind DKind) {
- return DKind == OMPD_taskloop || DKind == OMPD_taskloop_simd ||
- DKind == OMPD_master_taskloop || DKind
https://github.com/kparzysz edited
https://github.com/llvm/llvm-project/pull/83625
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kparzysz edited
https://github.com/llvm/llvm-project/pull/83625
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kparzysz updated
https://github.com/llvm/llvm-project/pull/83625
>From b62919c2ce24feb3c75a5bbecce3d6b6ee8e5b7e Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek
Date: Tue, 16 Jan 2024 16:40:47 -0600
Subject: [PATCH 1/4] [Frontend] Add leaf constructs and association to
Ope
@@ -231,6 +244,8 @@ static void EmitDirectivesDecl(RecordKeeper &Records,
raw_ostream &OS) {
OS << "bool isAllowedClauseForDirective(Directive D, "
<< "Clause C, unsigned Version);\n";
OS << "\n";
+ OS << "const llvm::SmallVector &getLeafConstructs(Directive
D);\n";
https://github.com/kparzysz updated
https://github.com/llvm/llvm-project/pull/83625
>From b62919c2ce24feb3c75a5bbecce3d6b6ee8e5b7e Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek
Date: Tue, 16 Jan 2024 16:40:47 -0600
Subject: [PATCH 1/5] [Frontend] Add leaf constructs and association to
Ope
@@ -665,60 +617,45 @@ bool
clang::isOpenMPTargetDataManagementDirective(OpenMPDirectiveKind DKind) {
}
bool clang::isOpenMPNestingTeamsDirective(OpenMPDirectiveKind DKind) {
- return DKind == OMPD_teams || DKind == OMPD_teams_distribute ||
- DKind == OMPD_teams_distr
https://github.com/kparzysz edited
https://github.com/llvm/llvm-project/pull/83625
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -231,6 +244,8 @@ static void EmitDirectivesDecl(RecordKeeper &Records,
raw_ostream &OS) {
OS << "bool isAllowedClauseForDirective(Directive D, "
<< "Clause C, unsigned Version);\n";
OS << "\n";
+ OS << "const llvm::SmallVector &getLeafConstructs(Directive
D);\n";
@@ -231,6 +244,8 @@ static void EmitDirectivesDecl(RecordKeeper &Records,
raw_ostream &OS) {
OS << "bool isAllowedClauseForDirective(Directive D, "
<< "Clause C, unsigned Version);\n";
OS << "\n";
+ OS << "const llvm::SmallVector &getLeafConstructs(Directive
D);\n";
1 - 100 of 388 matches
Mail list logo
