jplehr wrote:
Hi, this turned a few bots red.
Can this be fixed forward or should we revert in the meantime?
Let me know if you need help with the bot setup / build config. Thanks!
https://github.com/llvm/llvm-project/pull/134174
___
cfe-commits mailin
https://github.com/jplehr approved this pull request.
LGTM
Thanks!
https://github.com/llvm/llvm-project/pull/136317
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jplehr wrote:
I can try later today to get dumps/preprocessed source or maybe some better
idea on how to reproduce the issue.
@jhuber6 if you have immediate thoughts on how to provide that (or maybe even
what may be the issue), let me know.
https://github.com/llvm/llvm-project/pull/136128
jplehr wrote:
I think this broke one of our bots:
https://lab.llvm.org/buildbot/#/builders/10/builds/3726/steps/10/logs/stdio
The bot was red for another issue at the time, but the stack trace includes
methods that are touched in this PR.
This is part of the stack trace:
```
#12 0x748dc31b
jplehr wrote:
This should be device libs from ROCm 6.3.3.
```
$ apt show rocm-device-libs
Package: rocm-device-libs
Version: 1.0.0.60303-74~22.04
Priority: optional
Section: devel
Maintainer: ROCm Compiler Support
Installed-Size: 3424 kB
Depends: rocm-core
Homepage: https://github.com/RadeonOpe
jplehr wrote:
> LLVM Buildbot has detected a new failure on builder `clang-hip-vega20`
> running on `hip-vega20-0` while building `clang` at step 3 "annotate".
>
> Full details are available at:
> https://lab.llvm.org/buildbot/#/builders/123/builds/17410
>
> Here is the relevant piece of the
https://github.com/jplehr approved this pull request.
LGTM
thx
https://github.com/llvm/llvm-project/pull/134656
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jplehr wrote:
> Hi, I think this broke two of our buildbots -- for reasons I don't quite
> understand, but reverting fixed the issue locally. Can you please take a look
> at it?
>
> https://lab.llvm.org/buildbot/#/builders/10/builds/980
> https://lab.llvm.org/buildbot/#/builders/73/builds/143
jplehr wrote:
Hi, I think this broke two of our buildbots -- for reasons I don't quite
understand, but reverting fixed the issue locally.
Can you please take a look at it?
https://lab.llvm.org/buildbot/#/builders/10/builds/980
https://lab.llvm.org/buildbot/#/builders/73/builds/14304
https://gi
https://github.com/jplehr approved this pull request.
With the explanation that change seems reasonable. Thank you.
https://github.com/llvm/llvm-project/pull/129118
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/
jplehr wrote:
> I think people are just confusing what `-flto` means when put on a link job.
Including me. :)
Apologies for my ignorance, what does this do and why do we want it?
It allows for some (LTO-related?) flags to be forwarded/handled correctly?
https://github.com/llvm/llvm-project/pul
jplehr wrote:
> but it turns on LTO by default right?
This is very much how I read that patch too.
Maybe the flag names and comments are a bit misleading here.
https://github.com/llvm/llvm-project/pull/129118
___
cfe-commits mailing list
cfe-commits@l
https://github.com/jplehr approved this pull request.
Thanks for all the comments and addressing concerns.
https://github.com/llvm/llvm-project/pull/125648
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/l
@@ -932,9 +932,18 @@ def W_Joined : Joined<["-"], "W">, Group,
def Xanalyzer : Separate<["-"], "Xanalyzer">,
HelpText<"Pass to the static analyzer">, MetaVarName<"">,
Group;
-def Xarch__ : JoinedAndSeparate<["-"], "Xarch_">, Flags<[NoXarchOption]>,
- HelpText<"Pass to th
https://github.com/jplehr edited
https://github.com/llvm/llvm-project/pull/127890
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -932,9 +932,18 @@ def W_Joined : Joined<["-"], "W">, Group,
def Xanalyzer : Separate<["-"], "Xanalyzer">,
HelpText<"Pass to the static analyzer">, MetaVarName<"">,
Group;
-def Xarch__ : JoinedAndSeparate<["-"], "Xarch_">, Flags<[NoXarchOption]>,
- HelpText<"Pass to th
@@ -932,9 +932,18 @@ def W_Joined : Joined<["-"], "W">, Group,
def Xanalyzer : Separate<["-"], "Xanalyzer">,
HelpText<"Pass to the static analyzer">, MetaVarName<"">,
Group;
-def Xarch__ : JoinedAndSeparate<["-"], "Xarch_">, Flags<[NoXarchOption]>,
- HelpText<"Pass to th
https://github.com/jplehr commented:
Certainly an improvement. I think this is good, but do not feel confident right
now to already approve.
https://github.com/llvm/llvm-project/pull/127890
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https
jplehr wrote:
Did anybody to run the OpenMP VV suite before and after this change? Curious to
know if that improves results.
https://github.com/llvm/llvm-project/pull/125648
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.or
https://github.com/jplehr closed
https://github.com/llvm/llvm-project/pull/126495
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jplehr created
https://github.com/llvm/llvm-project/pull/126495
Reverts llvm/llvm-project#126297
Broken buildbots
https://lab.llvm.org/staging/#/builders/105/builds/15554
https://lab.llvm.org/buildbot/#/builders/30/builds/15490
Error is
```
# .---command stderr
#
jplehr wrote:
This broke our flang-enabled buildbot
https://lab.llvm.org/staging/#/builders/105/builds/15554
Looking at it currently
https://github.com/llvm/llvm-project/pull/126297
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists
@@ -24,11 +24,24 @@ namespace offloading {
/// This is the record of an object that just be registered with the offloading
/// runtime.
struct EntryTy {
+ /// Reserved bytes used to detect an older version of the struct, always
zero.
+ uint64_t Reserved = 0x0;
+ /// The cur
https://github.com/jplehr approved this pull request.
Fixes a build issue I was hitting in
https://github.com/llvm/llvm-project/pull/123673
https://github.com/llvm/llvm-project/pull/124116
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:
jplehr wrote:
Thank you.
https://github.com/llvm/llvm-project/pull/95474
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jplehr wrote:
Can someone more familiar with this push a fix? That would be much appreciated.
Thanks!
https://github.com/llvm/llvm-project/pull/120327
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listi
jplehr wrote:
Hi @tahonermann do you have an idea on why this is failing?
https://github.com/llvm/llvm-project/pull/120327
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jplehr wrote:
> Yep already working on it, integration issue with another change from me, my
> bad
Thank you for working on a fix. :)
https://github.com/llvm/llvm-project/pull/120266
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lis
jplehr wrote:
This also failed one of our bots:
https://lab.llvm.org/buildbot/#/builders/140/builds/14350
https://github.com/llvm/llvm-project/pull/120266
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/l
jplehr wrote:
> Do you mean adding a dedicate buildbot for the `offload` sub-project? I think
> the answer is yes. Actually we plan to setup more buildbots for any other
> sub-projects, including `flang`, `lld` and `openmp`, etc.
Not necessarily a dedicated buildbot for the `offload` project,
jplehr wrote:
Is there the possibility to set up a buildbot for this architecture?
https://github.com/llvm/llvm-project/pull/120173
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jplehr wrote:
> LLVM Buildbot has detected a new failure on builder
> `openmp-offload-amdgpu-runtime` running on `omp-vega20-0` while building
> `clang` at step 7 "Add check check-offload".
>
> Full details are available at:
> https://lab.llvm.org/buildbot/#/builders/30/builds/11206
>
> Here
jplehr wrote:
I believe this results in tests failures on our bot:
https://lab.llvm.org/staging/#/builders/105/builds/9796 (and a few others)
https://github.com/llvm/llvm-project/pull/117199
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http
https://github.com/jplehr approved this pull request.
LGTM to temporarily fix the bots.
https://github.com/llvm/llvm-project/pull/115172
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jplehr wrote:
Just out of curiosity: Are all these things documented reasonably well
somewhere?
https://github.com/llvm/llvm-project/pull/114401
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cf
jplehr wrote:
> I just pushed a fix in
> [75252e2](https://github.com/llvm/llvm-project/commit/75252e29ea6a0959f3c1670e641a03fc18fc65fa).
I see another error in one of our bots for this patch:
https://lab.llvm.org/staging/#/builders/130/builds/7112
https://github.com/llvm/llvm-project/pull/11
jplehr wrote:
Hi,
This broke our OpenMP / Offload bots. It would be great if that can be easily
fixed or reverted if more time is required to inspect the issue.
Thanks!
https://github.com/llvm/llvm-project/pull/113614
___
cfe-commits mailing list
cfe-
https://github.com/jplehr closed
https://github.com/llvm/llvm-project/pull/113548
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jplehr approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/113548
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jplehr edited
https://github.com/llvm/llvm-project/pull/112930
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jplehr approved this pull request.
Only a nit.
Let's see what happens.
https://github.com/llvm/llvm-project/pull/112930
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi
@@ -150,21 +151,30 @@ class MProtectHelper {
namespace {
-bool patchSled(const XRaySledEntry &Sled, bool Enable,
- int32_t FuncId) XRAY_NEVER_INSTRUMENT {
+bool isObjectLoaded(int32_t ObjId) {
+ SpinMutexLock Guard(&XRayInstrMapMutex);
+ if (ObjId < 0 || stati
jplehr wrote:
Would there be any reason to put entries for different offloading languages
into distinctly named “sub entries”?
https://github.com/llvm/llvm-project/pull/111890
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm
https://github.com/jplehr closed
https://github.com/llvm/llvm-project/pull/111978
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jplehr approved this pull request.
LGTM
next time please separate PRs for fix and formatting.
https://github.com/llvm/llvm-project/pull/111978
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/ma
https://github.com/jplehr closed https://github.com/llvm/llvm-project/pull/90959
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -9058,257 +9058,65 @@ void CGOpenMPRuntime::emitUserDefinedMapper(const
OMPDeclareMapperDecl *D,
return;
ASTContext &C = CGM.getContext();
QualType Ty = D->getType();
- QualType PtrTy = C.getPointerType(Ty).withRestrict();
- QualType Int64Ty = C.getIntTypeForBitwi
https://github.com/jplehr edited
https://github.com/llvm/llvm-project/pull/110001
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jplehr commented:
There is a nit, but I'm also not very familiar with this piece of code.
https://github.com/llvm/llvm-project/pull/110001
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman
jplehr wrote:
Thank you. Will take a closer look next week.
So far, I ran this through one of our buildbot configs and did not see an issue
there.
https://github.com/llvm/llvm-project/pull/110001
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/jplehr approved this pull request.
I think this is a reasonable change.
https://github.com/llvm/llvm-project/pull/109152
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm
jplehr wrote:
I ran this through a buildbot config and found no errors.
https://github.com/llvm/llvm-project/pull/109152
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jplehr wrote:
I guess that makes sense.
https://github.com/llvm/llvm-project/pull/109152
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jplehr wrote:
Hi, we see the same errors on our buildbots, two as examples:
https://lab.llvm.org/buildbot/#/builders/30/builds/6250
https://lab.llvm.org/buildbot/#/builders/140/builds/6648
https://github.com/llvm/llvm-project/pull/108627
___
cfe-commi
@@ -1533,6 +1565,67 @@ Error GenericDeviceTy::printInfo() {
return Plugin::success();
}
+Expected GenericDeviceTy::getKernel(llvm::StringRef Name,
+ DeviceImageTy *ImagePtr,
+
https://github.com/jplehr commented:
I tried a bot-config build and it fails with a compiler error, see my comment.
https://github.com/llvm/llvm-project/pull/104168
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/
https://github.com/jplehr edited
https://github.com/llvm/llvm-project/pull/104168
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jplehr wrote:
I did a few local run of this with our buildbot config and that appeared clean
for the time being.
https://github.com/llvm/llvm-project/pull/102691
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/ma
jplehr wrote:
For some reason this broke the bots, e.g.,
https://lab.llvm.org/buildbot/#/builders/30/builds/4417
https://lab.llvm.org/staging/#/builders/97/builds/2453
https://github.com/llvm/llvm-project/pull/104790
___
cfe-commits mailing list
cfe-c
https://github.com/jplehr commented:
Do we have some sort of documentation where this change of behavior needs to be
communicated?
https://github.com/llvm/llvm-project/pull/101032
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.l
jplehr wrote:
I was mostly curious if we have some coverage that would ideally break if the
implementation breaks.
https://github.com/llvm/llvm-project/pull/98940
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/m
jplehr wrote:
Hey @arsenm this broke all AMDGPU OpenMP Offload buildbots (e.g.,
https://lab.llvm.org/buildbot/#/builders/30).
Any chance you can fix these issues?
https://github.com/llvm/llvm-project/pull/83131
___
cfe-commits mailing list
cfe-commit
jplehr wrote:
Thank you @nikic.
I'll see to reproduce locally and narrow down as much as possible to provide
small reproducer.
https://github.com/llvm/llvm-project/pull/95126
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.
jplehr wrote:
I believe this broke our flang+openmp+offload bot:
https://lab.llvm.org/staging/#/builders/140/builds/10168
Happy to help looking into it.
https://github.com/llvm/llvm-project/pull/95126
___
cfe-commits mailing list
cfe-commits@lists.llv
jplehr wrote:
Should the NFCI changes (like initializing struct fields) be put into a
separate PR?
https://github.com/llvm/llvm-project/pull/94549
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/
@@ -1199,7 +1244,9 @@ llvm::Function *CGNVCUDARuntime::finalizeModule() {
}
return nullptr;
}
- if (CGM.getLangOpts().OffloadingNewDriver && RelocatableDeviceCode)
+ if (CGM.getLangOpts().OffloadViaLLVM)
+createOffloadingEntries();
+ else if (CGM.getLangOpts().
@@ -111,6 +156,71 @@ void __xray_init() XRAY_NEVER_INSTRUMENT {
#endif
}
+// Default visibility is hidden, so we have to explicitly make it visible to
+// DSO.
+SANITIZER_INTERFACE_ATTRIBUTE int32_t __xray_register_dso(
+const XRaySledEntry *SledsBegin, const XRaySledEntry
@@ -80,29 +140,14 @@ void __xray_init() XRAY_NEVER_INSTRUMENT {
return;
}
- {
-SpinMutexLock Guard(&XRayInstrMapMutex);
-XRayInstrMap.Sleds = __start_xray_instr_map;
-XRayInstrMap.Entries = __stop_xray_instr_map - __start_xray_instr_map;
-if (__start_xra
@@ -50,14 +52,72 @@ atomic_uint8_t XRayInitialized{0};
// This should always be updated before XRayInitialized is updated.
SpinMutex XRayInstrMapMutex;
-XRaySledMap XRayInstrMap;
+// XRaySledMap XRayInstrMap;
+// Contains maps for the main executable as well as DSOs.
+// std:
@@ -50,14 +52,72 @@ atomic_uint8_t XRayInitialized{0};
// This should always be updated before XRayInitialized is updated.
SpinMutex XRayInstrMapMutex;
-XRaySledMap XRayInstrMap;
+// XRaySledMap XRayInstrMap;
+// Contains maps for the main executable as well as DSOs.
+// std:
https://github.com/jplehr commented:
Thank you for contributing this patch. I am by no means an expert in this area,
but would like to see xray get support for shared libraries.
I did an initial pass to simply look for easy things and left a few comments.
https://github.com/llvm/llvm-project/p
@@ -111,6 +156,71 @@ void __xray_init() XRAY_NEVER_INSTRUMENT {
#endif
}
+// Default visibility is hidden, so we have to explicitly make it visible to
+// DSO.
+SANITIZER_INTERFACE_ATTRIBUTE int32_t __xray_register_dso(
+const XRaySledEntry *SledsBegin, const XRaySledEntry
https://github.com/jplehr edited https://github.com/llvm/llvm-project/pull/90959
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jplehr approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/88071
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -16,4 +16,12 @@ typedef unsigned __INTPTR_TYPE__ uintptr_t;
#error Every target should have __INTPTR_TYPE__
#endif
+#ifdef __INTPTR_MAX__
+#define INTPTR_MAX__INTPTR_MAX__
+#endif
+
+#ifdef __UINTPTR_MAX__
+#define UINTPTR_MAX __UINTPTR_MAX__
+#endif
+
--
https://github.com/jplehr commented:
The changes seem reasonable to me.
https://github.com/llvm/llvm-project/pull/88007
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jplehr edited https://github.com/llvm/llvm-project/pull/88007
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Stefan =?utf-8?q?Gr=C3=A4nitz?=
Message-ID:
In-Reply-To:
jplehr wrote:
I see. Thanks for the explanation and the fix! Bot is back to green.
https://github.com/llvm/llvm-project/pull/84461
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https
Stefan =?utf-8?q?Gränitz?=
Message-ID:
In-Reply-To:
jplehr wrote:
Thanks! I am quite unfamiliar with that part of the code base and wonder if the
symbol needs to just exist somewhere.
The other thing used there (`InitializeNativeTargetAsmPrinter`) is declared in
`TargetSelect.h`. So, does it
Stefan =?utf-8?q?Gränitz?=
Message-ID:
In-Reply-To:
jplehr wrote:
Hi,
I think this one broke one of our buildbots:
https://lab.llvm.org/buildbot/#/builders/259/builds/1769
I'm happy to help looking into it.
https://github.com/llvm/llvm-project/pull/84461
https://github.com/jplehr approved this pull request.
LG
https://github.com/llvm/llvm-project/pull/82699
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jplehr commented:
I looked at the changes and from the little I understand CMake they seem ok.
I added one nit.
Maybe @saiislam can have a look as well.
https://github.com/llvm/llvm-project/pull/81921
___
cfe-commits mailing list
cf
https://github.com/jplehr edited https://github.com/llvm/llvm-project/pull/81921
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -102,6 +80,121 @@ function(add_entrypoint_library target_name)
list(APPEND all_deps ${entrypoint_target})
endforeach(dep)
list(REMOVE_DUPLICATES all_deps)
+ set(${result} ${all_deps} PARENT_SCOPE)
+endfunction()
+
+# A rule to build a library from a collection of en
jplehr wrote:
This introduced a spelling mistake that broke some builds.
https://lab.llvm.org/buildbot/#/builders/193/builds/46220
https://github.com/llvm/llvm-project/pull/80162
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llv
jplehr wrote:
Hi @aaupov I think this did break the AMD Hip build bot (another annotated
builder).
Are you looking into the potential issue?
https://github.com/llvm/llvm-project/pull/69133
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:
https://github.com/jplehr closed https://github.com/llvm/llvm-project/pull/76571
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jplehr updated
https://github.com/llvm/llvm-project/pull/76571
>From a6c437a52674613b90c451c2ed4105265f420a32 Mon Sep 17 00:00:00 2001
From: JP Lehr
Date: Fri, 29 Dec 2023 04:32:24 -0500
Subject: [PATCH] [OpenMP][USM] Introduces -fopenmp-force-usm flag
This flag forces the c
jplehr wrote:
I just realized that I need to update the clang lit tests, so this is *not
ready to land*, but I don't see a button to indicate that.
https://github.com/llvm/llvm-project/pull/76571
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/jplehr updated
https://github.com/llvm/llvm-project/pull/76571
>From 41b227e2c84b3c7eeedb6a9ebf559bec2c34aec3 Mon Sep 17 00:00:00 2001
From: JP Lehr
Date: Fri, 29 Dec 2023 04:32:24 -0500
Subject: [PATCH] [OpenMP][USM] Introduces -fopenmp-force-usm flag
This flag forces the c
jplehr wrote:
> Automatic zero-copy doesn't work on some of the bbot's. I will have to land
> this once the lit test harness extension in #77851 re-lands.
Having your work landed would be very helpful indeed.
https://github.com/llvm/llvm-project/pull/76571
_
jplehr wrote:
@carlobertolli can you have another look at the runtime test I added to see if
that addresses your feedback?
https://github.com/llvm/llvm-project/pull/76571
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/c
https://github.com/jplehr updated
https://github.com/llvm/llvm-project/pull/76571
>From bf25a538e7c020efde557b595eba64b804cbb817 Mon Sep 17 00:00:00 2001
From: JP Lehr
Date: Fri, 29 Dec 2023 04:32:24 -0500
Subject: [PATCH 1/5] [OpenMP][USM] Introduces -fopenmp-force-usm flag
This flag forces t
https://github.com/jplehr updated
https://github.com/llvm/llvm-project/pull/76571
>From bf25a538e7c020efde557b595eba64b804cbb817 Mon Sep 17 00:00:00 2001
From: JP Lehr
Date: Fri, 29 Dec 2023 04:32:24 -0500
Subject: [PATCH 1/4] [OpenMP][USM] Introduces -fopenmp-force-usm flag
This flag forces t
https://github.com/jplehr closed https://github.com/llvm/llvm-project/pull/75467
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jplehr wrote:
Closing this. Test is now part of feature-PR.
https://github.com/llvm/llvm-project/pull/75467
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jplehr wrote:
While I add some documentation, I'd appreciate feedback especially on the lit
side of things. I would very much like to rename the pretty happy tripple-X
workaround for substitution debugging into something sane.
https://github.com/llvm/llvm-project/pull/76571
___
https://github.com/jplehr updated
https://github.com/llvm/llvm-project/pull/76571
>From bf25a538e7c020efde557b595eba64b804cbb817 Mon Sep 17 00:00:00 2001
From: JP Lehr
Date: Fri, 29 Dec 2023 04:32:24 -0500
Subject: [PATCH 1/3] [OpenMP][USM] Introduces -fopenmp-force-usm flag
This flag forces t
jplehr wrote:
We have the buildbot now up in staging:
https://lab.llvm.org/staging/#/builders/188
https://github.com/llvm/llvm-project/pull/71045
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/c
jplehr wrote:
The IR is impacted for the global that is in that test case. Lines ~46-50
(first IR section) vs line ~68 (second IR section). The remaining code is
indeed the same.
The way that this test is executed is out of date however, given that I have
reimplemented the flag.
My plan is to
1 - 100 of 132 matches
Mail list logo
