https://github.com/yln approved this pull request.
Thanks, approved with a few nits!
https://github.com/llvm/llvm-project/pull/86220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -15,6 +15,7 @@ endif()
# Must go below project(..)
include(GNUInstallDirs)
+include(GetDarwinLinkerVersion)
yln wrote:
Thanks for extracting this into a separate file!
https://github.com/llvm/llvm-project/pull/86220
@@ -51,6 +51,7 @@ set_default("expensive_checks",
@LLVM_ENABLE_EXPENSIVE_CHECKS_PYBOOL@)
set_default("test_standalone_build_libs",
@COMPILER_RT_TEST_STANDALONE_BUILD_LIBS_PYBOOL@)
set_default("has_compiler_rt_libatomic",
@COMPILER_RT_BUILD_STANDALONE_LIBATOMIC_PYBOOL@)
llvmbot wrote:
@llvm/pr-subscribers-hlsl
Author: Chris B (llvm-beanz)
Changes
This change just enables the SPIR-V target by default in the HLSL build
configurations. Since SPIR-V support is something we expect from the full HLSL
compiler releases for pairity with DXC we should enable it
https://github.com/llvm-beanz created
https://github.com/llvm/llvm-project/pull/86323
This change just enables the SPIR-V target by default in the HLSL build
configurations. Since SPIR-V support is something we expect from the full HLSL
compiler releases for pairity with DXC we should enable
rnk wrote:
I think it makes sense to remove carriage returns on checkin, but I'm not sure
it makes sense to add them back on checkout on Windows. Historically, it's been
really easy to write LLVM tests that fail when the source is checked out with
CRLF. Many Windows developers have noted that
@@ -444,6 +445,15 @@ else()
set(SANITIZER_USE_SYMBOLS FALSE)
endif()
+# Get the linker version while configuring compiler-rt and explicitly pass it
+# in cflags during testing. This fixes the compiler/linker version mismatch
+# issue when running a clang built with a newer
@@ -346,20 +347,7 @@ endif ()
# Determine HOST_LINK_VERSION on Darwin.
set(HOST_LINK_VERSION)
if (APPLE AND NOT CMAKE_LINKER MATCHES ".*lld.*")
- set(LD_V_OUTPUT)
- execute_process(
-COMMAND sh -c "${CMAKE_LINKER} -v 2>&1 | head -1"
-RESULT_VARIABLE HAD_ERROR
-
https://github.com/yln edited https://github.com/llvm/llvm-project/pull/86220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rayroudc updated
https://github.com/llvm/llvm-project/pull/86254
>From fa7abefb713ee7573ba2c48ecea55ed6ac2e5c59 Mon Sep 17 00:00:00 2001
From: "C. Rayroud"
Date: Mon, 18 Mar 2024 06:39:26 +
Subject: [PATCH] [clang-format] Fix anonymous reference parameter with default
adrian-prantl wrote:
While I'm sure this commit will manage to break _something_ unexpectedly, I
think this is reasonable to do! Thanks for taking this on.
https://github.com/llvm/llvm-project/pull/86318
___
cfe-commits mailing list
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 cd8286a667d568c4319b09baa63ba899e3101a19
2dfda2e4b11c7ea0a81dd4dcd43aa426039d1e0a --
ldrumm wrote:
I'm sure there are lots of people that will want to look at this, so please
bring in reviewers at will
https://github.com/llvm/llvm-project/pull/86318
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
danakj wrote:
The subspace library also provides mechanisms [for
querying](https://suslib.cc/sus-mem-TriviallyRelocatable.html) and marking
types trivially relocatable, and relies on the compiler's
`__is_trivially_relocatable` as a signal as well. It then optimizes containers
and operations
llvmbot wrote:
@llvm/pr-subscribers-backend-x86
Author: None (ldrumm)
Changes
Historically, we've not automatically enforced how git tracks line endings, but
there are many, many commits that "undo" unintended CRLFs getting into history.
`git log --pretty=oneline --grep=CRLF` shows
llvmbot wrote:
@llvm/pr-subscribers-clangd
@llvm/pr-subscribers-lldb
Author: None (ldrumm)
Changes
Historically, we've not automatically enforced how git tracks line endings, but
there are many, many commits that "undo" unintended CRLFs getting into history.
`git log --pretty=oneline
@@ -109,9 +109,22 @@ int unused_with_implicit_default_def(void) { return 1; }
int unused_with_implicit_forward_default_def(void) { return 0; }
__attribute__((target_version("lse"))) int
unused_with_implicit_forward_default_def(void) { return 1; }
-// This should generate a
@@ -109,9 +109,22 @@ int unused_with_implicit_default_def(void) { return 1; }
int unused_with_implicit_forward_default_def(void) { return 0; }
__attribute__((target_version("lse"))) int
unused_with_implicit_forward_default_def(void) { return 1; }
-// This should generate a
changpeng wrote:
> > global_load_re_b64
>
> Type global_load_re_b64.
Changed! Thanks.
https://github.com/llvm/llvm-project/pull/86313
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/changpeng edited
https://github.com/llvm/llvm-project/pull/86313
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rampitec wrote:
> global_load_re_b64
Type global_load_re_b64.
https://github.com/llvm/llvm-project/pull/86313
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Changpeng Fang (changpeng)
Changes
Rename the intrinsics to close to the instruction mnemonic names:
Use global_load_re_b64 and global_load_tr_b128 instead of global_load_tr.
This patch also removes f16/bf16 versions of
@@ -439,82 +444,247 @@
CGRecordLowering::accumulateBitFields(RecordDecl::field_iterator Field,
Members.push_back(MemberInfo(bitsToCharUnits(StartBitOffset),
MemberInfo::Field, nullptr, *Field));
}
-return;
+return Field;
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Changpeng Fang (changpeng)
Changes
Rename the intrinsics to close to the instruction mnemonic names:
Use global_load_re_b64 and global_load_tr_b128 instead of global_load_tr.
This patch also removes f16/bf16 versions of
https://github.com/changpeng created
https://github.com/llvm/llvm-project/pull/86313
Rename the intrinsics to close to the instruction mnemonic names:
Use global_load_re_b64 and global_load_tr_b128 instead of global_load_tr.
This patch also removes f16/bf16 versions of builtins/intrinsics.
Abhinkop wrote:
@amy-kwan I'm looking at it now. I will most probably raise a review some time
later in the evening or tomorrow if I am not able to complete it today.
https://github.com/llvm/llvm-project/pull/86131
___
cfe-commits mailing list
@@ -483,14 +483,16 @@ void just_fine(void) {}
__arm_locally_streaming
__attribute__((target_version("sme2")))
-void just_fine_locally_streaming(void) {}
+void incompatible_locally_streaming(void) {}
+// expected-error@-1 {{attribute 'target_version' multiversioning cannot be
ms178 wrote:
This patch also resolves my reported issue over at ClangBuiltLinux for
compiling the Linux Kernel on a recent LLVM/Clang snapshot.
https://github.com/llvm/llvm-project/pull/86017
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -109,9 +109,22 @@ int unused_with_implicit_default_def(void) { return 1; }
int unused_with_implicit_forward_default_def(void) { return 0; }
__attribute__((target_version("lse"))) int
unused_with_implicit_forward_default_def(void) { return 1; }
-// This should generate a
@@ -70,17 +69,23 @@ int __attribute__((target_version("lse"))) extc(void) {
return 1; }
auto __attribute__((target_version("default"))) ret1(void) { return 1; }
auto __attribute__((target_version("dpb"))) ret2(void) { return 1; }
+// expected-error@-1 {{attribute
Author: Aaron Ballman
Date: 2024-03-22T13:01:14-04:00
New Revision: 5184e6ad69b0ca69dfba6fb0982a675c595f49a2
URL:
https://github.com/llvm/llvm-project/commit/5184e6ad69b0ca69dfba6fb0982a675c595f49a2
DIFF:
https://github.com/llvm/llvm-project/commit/5184e6ad69b0ca69dfba6fb0982a675c595f49a2.diff
@@ -104,6 +103,7 @@ int __attribute__((target_version("sha2"))) combine(void) {
return 1; }
int __attribute__((aarch64_vector_pcs, target_version("sha3"))) combine(void)
{ return 2; }
int __attribute__((target_version("fp+aes+pmull+rcpc"))) unspec_args() {
return -1; }
+//
amy-kwan wrote:
> @amy-kwan I'm looking at it now. I will most probably raise a review some
> time later in the evening or tomorrow if I am not able to complete it today.
Sounds good, thank you so much, @Abhinkop!
https://github.com/llvm/llvm-project/pull/86131
@@ -70,17 +69,23 @@ int __attribute__((target_version("lse"))) extc(void) {
return 1; }
auto __attribute__((target_version("default"))) ret1(void) { return 1; }
auto __attribute__((target_version("dpb"))) ret2(void) { return 1; }
+// expected-error@-1 {{attribute
Author: Aaron Ballman
Date: 2024-03-22T12:50:18-04:00
New Revision: d2f684685afeffcffba7e889e7267bce1d905911
URL:
https://github.com/llvm/llvm-project/commit/d2f684685afeffcffba7e889e7267bce1d905911
DIFF:
https://github.com/llvm/llvm-project/commit/d2f684685afeffcffba7e889e7267bce1d905911.diff
@@ -68,13 +68,15 @@ int __attribute__((target_version(""))) unsup1(void) {
return 1; }
void __attribute__((target_version("crc32"))) unsup2(void) {}
void __attribute__((target_version("default+fp16"))) koo(void) {}
+//expected-error@-1 {{function multiversioning doesn't
@@ -483,14 +483,16 @@ void just_fine(void) {}
__arm_locally_streaming
__attribute__((target_version("sme2")))
-void just_fine_locally_streaming(void) {}
+void incompatible_locally_streaming(void) {}
+// expected-error@-1 {{attribute 'target_version' multiversioning cannot be
Author: Aaron Ballman
Date: 2024-03-22T12:50:18-04:00
New Revision: 527a624205748814dd9309eda7ee308b40b2359a
URL:
https://github.com/llvm/llvm-project/commit/527a624205748814dd9309eda7ee308b40b2359a
DIFF:
https://github.com/llvm/llvm-project/commit/527a624205748814dd9309eda7ee308b40b2359a.diff
jroelofs wrote:
re: stacking PRs: SPR is pretty handy: https://getcord.github.io/spr/
https://github.com/llvm/llvm-project/pull/85454
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DanielKristofKiss updated
https://github.com/llvm/llvm-project/pull/86212
>From 33d8277d188f82847d914273be2379151dd33d41 Mon Sep 17 00:00:00 2001
From: Daniel Kiss
Date: Fri, 8 Mar 2024 15:06:28 +0100
Subject: [PATCH 1/2] BTI,GCS,PAC Module flag update.
Module flag is used
https://github.com/yxsamliu closed
https://github.com/llvm/llvm-project/pull/85976
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Yaxun (Sam) Liu
Date: 2024-03-22T12:30:02-04:00
New Revision: 8155ec13968b6457c61b8507f2ae8ba3ac3b748b
URL:
https://github.com/llvm/llvm-project/commit/8155ec13968b6457c61b8507f2ae8ba3ac3b748b
DIFF:
https://github.com/wrotki approved this pull request.
https://github.com/llvm/llvm-project/pull/86220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
wrotki wrote:
Yes, it makes sense
https://github.com/llvm/llvm-project/pull/86220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/wangpc-pp updated
https://github.com/llvm/llvm-project/pull/83774
>From f1653805def59bc6eb23052b3ade80c900b4daaa Mon Sep 17 00:00:00 2001
From: wangpc
Date: Fri, 14 Jul 2023 10:38:14 +0800
Subject: [PATCH 1/2] [clang] Enable sized deallocation by default in C++14
onwards
@@ -1318,16 +1318,16 @@ define void @instructions.va_arg(i8* %v, ...) {
%ap2 = bitcast i8** %ap to i8*
call void @llvm.va_start(i8* %ap2)
- ; CHECK: call void @llvm.va_start(ptr %ap2)
+ ; CHECK: call void @llvm.va_start.p0(ptr %ap2)
va_arg i8* %ap2, i32
; CHECK:
https://github.com/jrtc27 approved this pull request.
LGTM once typo is fixed
https://github.com/llvm/llvm-project/pull/85460
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jrtc27 edited https://github.com/llvm/llvm-project/pull/85460
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
amy-kwan wrote:
Thanks @Abhinkop for the patch! I have approved the compiler-rt patch since it
does appear to resolve the check-runtimes/check-all issue for the asan test
case.
If you also would like me to test a follow up patch for
`llvm-project/llvm/lib/IR/Core.cpp`, I would be happy to do
https://github.com/arichardson approved this pull request.
LGTM thanks! Please wait for @jrtc27 to also approve before committing though.
https://github.com/llvm/llvm-project/pull/85460
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
balazske wrote:
It looks like that this change causes crashes on many projects (curl, vim,
postgres, others) in `RAIIMutexDescriptor::initIdentifierInfo`.
https://github.com/llvm/llvm-project/pull/80029
___
cfe-commits mailing list
@@ -4258,6 +4258,18 @@ bool CompilerInvocation::ParseLangArgs(LangOptions
, ArgList ,
} else {
llvm_unreachable("expected DXIL or SPIR-V target");
}
+ // validate that if fnative-half-type is given, that
+ // the language standard is at least
changpeng wrote:
I am going to propose to rename intrinsics and remove f16/bf16 versions of
builtins/intrinsics
https://github.com/llvm/llvm-project/pull/86202
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/changpeng closed
https://github.com/llvm/llvm-project/pull/86202
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Changpeng Fang
Date: 2024-03-22T08:51:53-07:00
New Revision: 3054d0dae7a813c493d2bb8e969aa2321145a83b
URL:
https://github.com/llvm/llvm-project/commit/3054d0dae7a813c493d2bb8e969aa2321145a83b
DIFF:
@@ -4258,6 +4258,18 @@ bool CompilerInvocation::ParseLangArgs(LangOptions
, ArgList ,
} else {
llvm_unreachable("expected DXIL or SPIR-V target");
}
+ // validate that if fnative-half-type is given, that
+ // the language standard is at least
Author: Wang Pengcheng
Date: 2024-03-22T23:21:11+08:00
New Revision: b44771f480385fa93ba7719a57e759e19747e709
URL:
https://github.com/llvm/llvm-project/commit/b44771f480385fa93ba7719a57e759e19747e709
DIFF:
changpeng wrote:
[AMD Official Use Only - General]
I am fine to remove f16/bf16 versions. Enumerating all possible types could be
very painful. For example we gave up enumerating for B64, and ended up using
v2i32 only. What do others think removing f16/bf16 versions? Thanks
Get Outlook for
https://github.com/arsenm approved this pull request.
https://github.com/llvm/llvm-project/pull/85460
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Abhinkop wrote:
@AaronBallman @amy-kwan Here is the pull request for compiler-rt patch
https://github.com/llvm/llvm-project/pull/86290. I am not able to add reviewers
to that pull request.
https://github.com/llvm/llvm-project/pull/86131
___
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/74440
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/hnakamura5 closed
https://github.com/llvm/llvm-project/pull/76059
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hnakamura5 wrote:
All the split parts of this PR is merged. Thank you!
https://github.com/llvm/llvm-project/pull/76059
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hnakamura5 wrote:
@HazardyKnusperkeks
Thank you very much!
It took about 4 month with ten or more PRs for the parts of the first concept
of TableGen formatting.
You reviewed every time and gave me many valuable suggestions. I appreciate you
again and again!
Author: Aaron Ballman
Date: 2024-03-22T10:17:50-04:00
New Revision: d231e3b10ead90e4360f7ceb88e4bca9d42d7d04
URL:
https://github.com/llvm/llvm-project/commit/d231e3b10ead90e4360f7ceb88e4bca9d42d7d04
DIFF:
https://github.com/llvm/llvm-project/commit/d231e3b10ead90e4360f7ceb88e4bca9d42d7d04.diff
https://github.com/hnakamura5 closed
https://github.com/llvm/llvm-project/pull/86150
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Hirofumi Nakamura
Date: 2024-03-22T23:11:36+09:00
New Revision: e54af608160350baa7ae1b8069f916eb625beadd
URL:
https://github.com/llvm/llvm-project/commit/e54af608160350baa7ae1b8069f916eb625beadd
DIFF:
https://github.com/AaronBallman commented:
Can you also add a test showing the difference between `-Wformat
-Wformat-signedness` and `-Wformat-signedness` by itself (which does nothing)?
I'd also like to see a test demonstrating that `#pragma GCC diagnostic ignored
-Wformat` disables the
@@ -12465,6 +12465,20 @@ isArithmeticArgumentPromotion(Sema , const
ImplicitCastExpr *ICE) {
S.Context.getFloatingTypeOrder(From, To) < 0;
}
+static analyze_format_string::ArgType::MatchKind
+handleFormatSignedness(analyze_format_string::ArgType::MatchKind Match,
+
https://github.com/MDevereau closed
https://github.com/llvm/llvm-project/pull/85070
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
NagyDonat wrote:
> Did you envision [debug] XYZ, and XYZ [-analyzer-note-analysis-entry-points]?
Which should I pursue?
I slightly prefer [debug] because this mode can be activated by two different
flags, but mentioning the flag is also fine.
(Sorry for not answering earlier, I didn't have a
@@ -11,20 +11,27 @@
// Basic test for float registers number are accepted.
-#include
#include
#include
#include
+// Using __attribute__((section("main_func"))) is Linux specific, but then
al45tair wrote:
Fair point.
wheatman wrote:
I made the changes, added the tests, and fixed some things that the tests found.
A few questions.
It seems like n3030 is only partially supported, specifically
```
extern enum forward fwd_val0; /* Constraint violation: incomplete type */
extern enum forward* fwd_ptr0; /*
Abhinkop wrote:
Yes, I will create a separate patch. I'm not sure it would be able to do it in
this patch. I am halfway through verifying the fix in this patch.
https://github.com/llvm/llvm-project/pull/86131
___
cfe-commits mailing list
AaronBallman wrote:
Oof, that one is a bit messier to resolve -- it's mixing unrelated reference
and pointer types and hoping the casts correctly convert back and forth
properly. The fix to that should probably be done separately from the sanitizer
fix so we get reasonable code review from
https://github.com/NagyDonat edited
https://github.com/llvm/llvm-project/pull/83675
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/NagyDonat requested changes to this pull request.
I'll try to take over this review process and help with finalizing this commit.
I also added @balazske who's also familiar with this area.
Unfortunately currently there are significant problems in the state/assumption
@@ -2515,6 +2517,53 @@ void CStringChecker::evalSprintfCommon(CheckerContext
, const CallEvent ,
C.addTransition(State);
}
+void CStringChecker::evalGetentropy(CheckerContext , const CallEvent )
const {
+ DestinationArgExpr Buffer = {{Call.getArgExpr(0), 0}};
+
@@ -2515,6 +2517,53 @@ void CStringChecker::evalSprintfCommon(CheckerContext
, const CallEvent ,
C.addTransition(State);
}
+void CStringChecker::evalGetentropy(CheckerContext , const CallEvent )
const {
+ DestinationArgExpr Buffer = {{Call.getArgExpr(0), 0}};
+
https://github.com/jhuber6 approved this pull request.
https://github.com/llvm/llvm-project/pull/85976
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -529,3 +529,37 @@ void nocrash_on_locint_offset(void *addr, void* from,
struct S s) {
size_t iAdd = (size_t) addr;
memcpy(((void *) &(s.f)), from, iAdd);
}
+
+//===--===//
+// getentropy()
@@ -529,3 +529,37 @@ void nocrash_on_locint_offset(void *addr, void* from,
struct S s) {
size_t iAdd = (size_t) addr;
memcpy(((void *) &(s.f)), from, iAdd);
}
+
+//===--===//
+// getentropy()
@@ -2515,6 +2517,53 @@ void CStringChecker::evalSprintfCommon(CheckerContext
, const CallEvent ,
C.addTransition(State);
}
+void CStringChecker::evalGetentropy(CheckerContext , const CallEvent )
const {
+ DestinationArgExpr Buffer = {{Call.getArgExpr(0), 0}};
+
@@ -2516,6 +2518,47 @@ void CStringChecker::evalSprintfCommon(CheckerContext
, const CallEvent ,
C.addTransition(State);
}
+void CStringChecker::evalGetentropy(CheckerContext ,
+const CallEvent ) const {
+ DestinationArgExpr Buffer =
@@ -2515,6 +2517,53 @@ void CStringChecker::evalSprintfCommon(CheckerContext
, const CallEvent ,
C.addTransition(State);
}
+void CStringChecker::evalGetentropy(CheckerContext , const CallEvent )
const {
+ DestinationArgExpr Buffer = {{Call.getArgExpr(0), 0}};
+
@@ -2515,6 +2517,53 @@ void CStringChecker::evalSprintfCommon(CheckerContext
, const CallEvent ,
C.addTransition(State);
}
+void CStringChecker::evalGetentropy(CheckerContext , const CallEvent )
const {
+ DestinationArgExpr Buffer = {{Call.getArgExpr(0), 0}};
+
@@ -2515,6 +2517,53 @@ void CStringChecker::evalSprintfCommon(CheckerContext
, const CallEvent ,
C.addTransition(State);
}
+void CStringChecker::evalGetentropy(CheckerContext , const CallEvent )
const {
+ DestinationArgExpr Buffer = {{Call.getArgExpr(0), 0}};
+
@@ -2515,6 +2517,53 @@ void CStringChecker::evalSprintfCommon(CheckerContext
, const CallEvent ,
C.addTransition(State);
}
+void CStringChecker::evalGetentropy(CheckerContext , const CallEvent )
const {
+ DestinationArgExpr Buffer = {{Call.getArgExpr(0), 0}};
+
@@ -2515,6 +2517,53 @@ void CStringChecker::evalSprintfCommon(CheckerContext
, const CallEvent ,
C.addTransition(State);
}
+void CStringChecker::evalGetentropy(CheckerContext , const CallEvent )
const {
+ DestinationArgExpr Buffer = {{Call.getArgExpr(0), 0}};
+
amy-kwan wrote:
I have also found one more that is not related to the sanitizers, but when
`llvm-project/llvm/lib/IR/Core.cpp`:
https://lab.llvm.org/buildbot/#/builders/36/builds/43840/steps/12/logs/stdio
Would it be possible to also resolve this one, as well?
https://github.com/yxsamliu updated
https://github.com/llvm/llvm-project/pull/85976
>From 1d14bcff6363b34ae48eac2bf68221b16dd1c855 Mon Sep 17 00:00:00 2001
From: "Yaxun (Sam) Liu"
Date: Wed, 20 Mar 2024 13:34:29 -0400
Subject: [PATCH] [HIP][NFC] Refactor managed var codegen
Refactor managed
Author: Amir Ayupov
Date: 2024-03-22T06:07:17-07:00
New Revision: 3b3de48fd84b8269d5f45ee0a9dc6b7448368424
URL:
https://github.com/llvm/llvm-project/commit/3b3de48fd84b8269d5f45ee0a9dc6b7448368424
DIFF:
https://github.com/llvm/llvm-project/commit/3b3de48fd84b8269d5f45ee0a9dc6b7448368424.diff
@@ -1160,9 +1152,8 @@ void CGNVCUDARuntime::createOffloadingEntries() {
// Returns module constructor to be added.
llvm::Function *CGNVCUDARuntime::finalizeModule() {
+ transformManagedVars();
yxsamliu wrote:
we do not have test for managed var in
https://github.com/yxsamliu edited
https://github.com/llvm/llvm-project/pull/85976
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1160,9 +1152,8 @@ void CGNVCUDARuntime::createOffloadingEntries() {
// Returns module constructor to be added.
llvm::Function *CGNVCUDARuntime::finalizeModule() {
+ transformManagedVars();
yxsamliu wrote:
we did the equivalent transformation previously
https://github.com/yronglin closed
https://github.com/llvm/llvm-project/pull/85263
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Paul Robinson
Date: 2024-03-22T05:36:41-07:00
New Revision: 04a6e0f1634f9a53120c27a30250d26dff4ada1c
URL:
https://github.com/llvm/llvm-project/commit/04a6e0f1634f9a53120c27a30250d26dff4ada1c
DIFF:
https://github.com/llvm/llvm-project/commit/04a6e0f1634f9a53120c27a30250d26dff4ada1c.diff
Abhinkop wrote:
Yes. I have done that. The tests failed because zlib was not enabled. Running
them with a rebuild after force-enabling zlib now. Will keep you updated on the
results.
https://github.com/llvm/llvm-project/pull/86131
___
cfe-commits
https://github.com/AaronBallman approved this pull request.
LGTM, do you need someone to land this on your behalf?
https://github.com/llvm/llvm-project/pull/78253
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/jhuber6 closed
https://github.com/llvm/llvm-project/pull/86217
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
101 - 200 of 260 matches
Mail list logo