rampitec wrote:
> I suppose left shift of negative values is undefined because if you shift out
> the sign bit you can overflow and get a positive value.
Sounds like BS. It is defined. Unexpected maybe.
https://github.com/llvm/llvm-project/pull/68740
___
mikaelholmen wrote:
I suppose left shift of negative values is undefined because if you shift out
the sign bit you can overflow and get a positive value.
https://github.com/llvm/llvm-project/pull/68740
___
cfe-commits mailing list
cfe-commits@lists.ll
rampitec wrote:
I have a small little problem that I cannot build tip now:
```
FAILED: tools/llvm-remarkutil/CMakeFiles/llvm-remarkutil.dir/RemarkCounter.cpp.o
CCACHE_CPP2=yes CCACHE_HASHDIR=yes /usr/bin/ccache /usr/bin/clang++
-DGTEST_HAS_RTTI=0 -D_DEBUG -D_GLIBCXX_ASSERTIONS -D_GNU_SOURCE
-D_
MaskRay wrote:
(Was out of town...)
As mentioned at
https://github.com/llvm/llvm-project/pull/67360#issuecomment-1736187374 , I
think a default error is probably not appropriate. A priority value <= 100 may
compete with a runtime `__attribute__((constructor(X)))` but more often it will
work.
rampitec wrote:
> > Hi @rampitec
> > With UBSan built binaries the MC/AMDGPU/literals.s testcase fails and
> > triggers UB like
> > ```
> > 07:33:04 ../lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp:2246:59:
> > runtime error: left shift of negative value -54321
> > 07:33:04 SUMMARY: Undefined
@@ -478,6 +480,43 @@ template class SourceLocScope final {
bool Enabled = false;
};
+template class StoredOpaqueValueScope final {
+public:
+ StoredOpaqueValueScope(ByteCodeExprGen *Ctx) : Ctx(Ctx) {}
+
+ bool VisitAndStoreOpaqueValue(const OpaqueValueExpr *Ove) {
+a
@@ -478,6 +480,43 @@ template class SourceLocScope final {
bool Enabled = false;
};
+template class StoredOpaqueValueScope final {
+public:
+ StoredOpaqueValueScope(ByteCodeExprGen *Ctx) : Ctx(Ctx) {}
+
+ bool VisitAndStoreOpaqueValue(const OpaqueValueExpr *Ove) {
+a
rampitec wrote:
> Hi @rampitec
>
> With UBSan built binaries the MC/AMDGPU/literals.s testcase fails and
> triggers UB like
>
> ```
> 07:33:04 ../lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp:2246:59: runtime
> error: left shift of negative value -54321
> 07:33:04 SUMMARY: UndefinedBehavior
@@ -2732,100 +2732,114 @@ static Instruction *matchFunnelShift(Instruction &Or,
InstCombinerImpl &IC) {
// rotate matching code under visitSelect and visitTrunc?
unsigned Width = Or.getType()->getScalarSizeInBits();
- // First, find an or'd pair of opposite shifts:
- //
@@ -2732,100 +2732,114 @@ static Instruction *matchFunnelShift(Instruction &Or,
InstCombinerImpl &IC) {
// rotate matching code under visitSelect and visitTrunc?
unsigned Width = Or.getType()->getScalarSizeInBits();
- // First, find an or'd pair of opposite shifts:
- //
tbaederr wrote:
https://reviews.llvm.org/D156565 hasn't been merged as far as I can tell and
it's the only test case I have for this. :(
https://github.com/llvm/llvm-project/pull/67990
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://li
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/68081
>From a7f0b32783bd8e9ce4d74d98e5a45351d3b38d24 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Tue, 3 Oct 2023 11:05:27 +0200
Subject: [PATCH] [clang][Interp] IntegralAP zero-init
---
clang/
mikaelholmen wrote:
Hi @rampitec
With UBSan built binaries the
MC/AMDGPU/literals.s
testcase fails and triggers UB like
```
07:33:04 ../lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp:2246:59: runtime
error: left shift of negative value -54321
07:33:04 SUMMARY: UndefinedBehaviorSanitizer: und
@@ -1642,9 +1642,9 @@ bool
ByteCodeExprGen::visitZeroInitializer(QualType QT,
case PT_Uint64:
return this->emitZeroUint64(E);
case PT_IntAP:
+return this->emitZeroIntAP(128, E); // FIXME: Ctx.getBitWidth()
case PT_IntAPS:
-assert(false);
-return false;
+
@@ -100,12 +100,13 @@ template class IntegralAP final {
}
static IntegralAP from(const Boolean &B) {
assert(false);
-return IntegralAP::zero();
+return IntegralAP::zero(1);
}
- static IntegralAP zero() {
-assert(false);
-return IntegralAP(0);
+ s
@@ -100,12 +100,13 @@ template class IntegralAP final {
}
static IntegralAP from(const Boolean &B) {
assert(false);
-return IntegralAP::zero();
+return IntegralAP::zero(1);
tbaederr wrote:
This is unimplemented right now, I just added the bitw
@@ -574,6 +574,20 @@ benchmarks)
generate-cmake
check-cxx-benchmarks
;;
+check-lldb-data-formatters)
+clean
+${CMAKE}\
+-S "${MONOREPO_ROOT}/llvm" \
+-B "${BUILD_DIR}"
@@ -574,6 +574,20 @@ benchmarks)
generate-cmake
check-cxx-benchmarks
;;
+check-lldb-data-formatters)
+clean
+${CMAKE}\
+-S "${MONOREPO_ROOT}/llvm" \
+-B "${BUILD_DIR}"
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/68679
>From 9598f33926ede7a35a431bebc3ed6f8ce20b03ef Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Tue, 10 Oct 2023 08:52:43 +0200
Subject: [PATCH] [clang][Interp] Support AddOffset with 128bit of
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/68878
>From b93096929aa98e17dfdb0240a9285d315fc95bfc Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 12 Oct 2023 19:31:08 +0800
Subject: [PATCH 1/2] [clang]Transform uninstantiated ExceptionSpec in
Template
llvmbot wrote:
@llvm/pr-subscribers-clang-modules
Author: Jan Svoboda (jansvoboda11)
Changes
This prevents redefinition errors due to having multiple paths for the same
module map. (rdar://24116019)
Originally implemented and tested downstream by @bcardosolopes, I just
made use of `File
https://github.com/jansvoboda11 created
https://github.com/llvm/llvm-project/pull/68957
This prevents redefinition errors due to having multiple paths for the same
module map. (rdar://24116019)
Originally implemented and tested downstream by @bcardosolopes, I just made use
of `FileEntryRef::g
yetingk wrote:
> The patch basically changes the ShadowCallStack back-end to emit an
> sspush/sspopchk instead of the usual SCS push/pop, which seems like a
> reasonable approach to me. However, it would be helpful to mention the
> dependency on `-fsanitize=shadow-call-stack` in the commit mes
https://github.com/yetingk updated
https://github.com/llvm/llvm-project/pull/68075
>From 91bb1d9884276a37f93515a648aa6ece353fdc70 Mon Sep 17 00:00:00 2001
From: Yeting Kuo
Date: Tue, 12 Sep 2023 12:28:00 +0800
Subject: [PATCH 1/4] [RISCV] Add MC layer support for Zicfiss.
The patch adds the in
https://github.com/KanRobert approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/68944
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2732,100 +2732,114 @@ static Instruction *matchFunnelShift(Instruction &Or,
InstCombinerImpl &IC) {
// rotate matching code under visitSelect and visitTrunc?
unsigned Width = Or.getType()->getScalarSizeInBits();
- // First, find an or'd pair of opposite shifts:
- //
@@ -2732,100 +2732,114 @@ static Instruction *matchFunnelShift(Instruction &Or,
InstCombinerImpl &IC) {
// rotate matching code under visitSelect and visitTrunc?
unsigned Width = Or.getType()->getScalarSizeInBits();
- // First, find an or'd pair of opposite shifts:
- //
https://github.com/yjijd created https://github.com/llvm/llvm-project/pull/68954
None
>From 74d9ea74aea28db6f5a3a4c6dfccb132d87d1fa8 Mon Sep 17 00:00:00 2001
From: licongtian
Date: Tue, 10 Oct 2023 15:51:58 +0800
Subject: [PATCH] [Clang][LoongArch] Add ABI implementation of passing vectors
---
https://github.com/yjijd created https://github.com/llvm/llvm-project/pull/68952
This patch adds compiler options -mlsx/-mlasx which enables the instruction
sets of LSX and LASX, and sets related predefined macros acording to the
options.
>From ba3d3560675bcfd84bd11bfed9e26ad86b60faaf Mon Sep
danix800 wrote:
Caused by missing `MemberSpecializationInfo` when importing so moving testcase
into `ASTImporterTest` as unittest would be better.
https://github.com/llvm/llvm-project/pull/68774
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -0,0 +1,57 @@
+# RUN: llvm-mc -triple=riscv64 -show-encoding
--mattr=+v,+xsfvqmaccqoq,+xsfvqmaccdod %s \
+# RUN:| FileCheck %s --check-prefixes=CHECK-ENCODING,CHECK-INST
+# RUN: not llvm-mc -triple=riscv64 -show-encoding %s 2>&1 \
+# RUN:| FileCheck %s --check-
@@ -642,18 +642,8 @@ get_sanitizer_flags(SANITIZER_FLAGS
"${LLVM_USE_SANITIZER}")
# Link system libraries ===
function(cxx_link_system_libraries target)
-
-# In order to remove just libc++ from the link step
-# we need to us
@@ -642,18 +642,8 @@ get_sanitizer_flags(SANITIZER_FLAGS
"${LLVM_USE_SANITIZER}")
# Link system libraries ===
function(cxx_link_system_libraries target)
-
-# In order to remove just libc++ from the link step
-# we need to us
https://github.com/ldionne updated
https://github.com/llvm/llvm-project/pull/68832
>From 9ca88d1a30ec1bcaa9c3c943c32649c36ad136d0 Mon Sep 17 00:00:00 2001
From: Louis Dionne
Date: Wed, 11 Oct 2023 12:21:39 -0700
Subject: [PATCH 1/3] [libc++] Use -nostdlib++ on GCC unconditionally
We support GC
https://github.com/ldionne updated
https://github.com/llvm/llvm-project/pull/68832
>From 9ca88d1a30ec1bcaa9c3c943c32649c36ad136d0 Mon Sep 17 00:00:00 2001
From: Louis Dionne
Date: Wed, 11 Oct 2023 12:21:39 -0700
Subject: [PATCH 1/3] [libc++] Use -nostdlib++ on GCC unconditionally
We support GC
@@ -20,6 +20,11 @@
#include
#endif
+#if !(defined(_MSC_VER) || defined(__SCE__)) || __has_feature(modules) ||
\
+defined(__UINTR__)
FreddyLeaf wrote:
3e32e495d8fa56ad3770769a5d4559ea0a41af96
https://github.com/llvm/llvm-project/pull/68944
@@ -922,8 +922,6 @@ endif()
include(HandleLLVMOptions)
-##
-
FreddyLeaf wrote:
3e32e495d8fa56ad3770769a5d4559ea0a41af96
https://github.com/llvm/llvm-project/pull/68944
___
cfe-commits mailing list
cfe-commits@
@@ -241,6 +241,7 @@ X86_FEATURE (SM3, "sm3")
X86_FEATURE (SM4, "sm4")
X86_FEATURE (AVXVNNIINT16,"avxvnniint16")
X86_FEATURE (EVEX512, "evex512")
+X86_FEATURE (USERMSR,"usermsr")
Fredd
@@ -0,0 +1,26 @@
+# RUN: llvm-mc --disassemble %s -triple=x86_64 | FileCheck %s
--check-prefixes=ATT
+# RUN: llvm-mc --disassemble %s -triple=x86_64 --output-asm-variant=1 |
FileCheck %s --check-prefixes=INTEL
+
+# ATT: urdmsr $123, %r9
+# INTEL: urdmsr r9, 123
+0xc4,0xc7,0x7b
@@ -325,6 +325,8 @@ def FeatureTSXLDTRK : SubtargetFeature<"tsxldtrk",
"HasTSXLDTRK", "true",
"Support TSXLDTRK instructions">;
def FeatureUINTR : SubtargetFeature<"uintr", "HasUINTR", "true",
"Has UINT
https://github.com/FreddyLeaf updated
https://github.com/llvm/llvm-project/pull/68944
>From 2377ab2b9865d8f152996fd38f6b543767f8c2ae Mon Sep 17 00:00:00 2001
From: Freddy Ye
Date: Wed, 11 Oct 2023 14:09:02 +0800
Subject: [PATCH 1/2] Add USER_MSR instructions.
For more details about this instru
@@ -0,0 +1,26 @@
+# RUN: llvm-mc --disassemble %s -triple=x86_64 | FileCheck %s
--check-prefixes=ATT
+# RUN: llvm-mc --disassemble %s -triple=x86_64 --output-asm-variant=1 |
FileCheck %s --check-prefixes=INTEL
+
+# ATT: urdmsr $123, %r9
+# INTEL: urdmsr r9, 123
+0xc4,0xc7,0x7b
htyu wrote:
> The performance win depends a lot on value distribution. For large copies,
> using SIMD with nontemporal hint is the way to go.
Right, and the dominating single-range distribution is also important for our
approach, similar to how speculative indirect call promotion works.
Rece
@@ -325,6 +325,8 @@ def FeatureTSXLDTRK : SubtargetFeature<"tsxldtrk",
"HasTSXLDTRK", "true",
"Support TSXLDTRK instructions">;
def FeatureUINTR : SubtargetFeature<"uintr", "HasUINTR", "true",
"Has UINT
@@ -0,0 +1,26 @@
+# RUN: llvm-mc --disassemble %s -triple=x86_64 | FileCheck %s
--check-prefixes=ATT
+# RUN: llvm-mc --disassemble %s -triple=x86_64 --output-asm-variant=1 |
FileCheck %s --check-prefixes=INTEL
+
+# ATT: urdmsr $123, %r9
+# INTEL: urdmsr r9, 123
+0xc4,0xc7,0x7b
@@ -241,6 +241,7 @@ X86_FEATURE (SM3, "sm3")
X86_FEATURE (SM4, "sm4")
X86_FEATURE (AVXVNNIINT16,"avxvnniint16")
X86_FEATURE (EVEX512, "evex512")
+X86_FEATURE (USERMSR,"usermsr")
phoeb
@@ -922,8 +922,6 @@ endif()
include(HandleLLVMOptions)
-##
-
phoebewang wrote:
Unrelated change
https://github.com/llvm/llvm-project/pull/68944
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://l
@@ -20,6 +20,11 @@
#include
#endif
+#if !(defined(_MSC_VER) || defined(__SCE__)) || __has_feature(modules) ||
\
+defined(__UINTR__)
phoebewang wrote:
__USERMSR__
https://github.com/llvm/llvm-project/pull/68944
_
https://github.com/NatashaKnk created
https://github.com/llvm/llvm-project/pull/68945
None
>From e431cc466e5609e19757481604dad47558219166 Mon Sep 17 00:00:00 2001
From: Natasha Kononenko
Date: Fri, 13 Oct 2023 00:48:17 +
Subject: [PATCH 1/2] Add ContractionOpInterface utility functions for
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 a712244f3b76cd2ef60b4f3ce5efaf6d4d49c6fe
2377ab2b9865d8f152996fd38f6b543767f8c2ae --
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Freddy Ye (FreddyLeaf)
Changes
For more details about this instruction, please refer to the latest ISE
document:
https://www.intel.com/content/www/us/en/develop/download/intel-architecture-instruction-set-extensions-programming-reference.
https://github.com/FreddyLeaf edited
https://github.com/llvm/llvm-project/pull/68944
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/FreddyLeaf created
https://github.com/llvm/llvm-project/pull/68944
For more details about this instruction, please refer to the latest ISE
document:
https://www.intel.com/content/www/us/en/develop/download/intel-architecture-instruction-set-extensions-programming-reference.h
Artem-B wrote:
@rnk -- would that be acceptable for clang-cl on windows?
https://github.com/llvm/llvm-project/pull/68921
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -827,6 +829,9 @@ bool ByteCodeExprGen::VisitArrayInitLoopExpr(
template
bool ByteCodeExprGen::VisitOpaqueValueExpr(const OpaqueValueExpr *E) {
+ if (OpaqueExprs.contains(E))
+return this->emitGetLocal(*classify(E), OpaqueExprs[E], E);
isuckatcs wrote:
@@ -176,3 +176,34 @@ Predefined Macros
* - ``HIP_API_PER_THREAD_DEFAULT_STREAM``
- Alias to ``__HIP_API_PER_THREAD_DEFAULT_STREAM__``. Deprecated.
+C++20 Concepts with HIP and CUDA
+
+
+In Clang, when working with HIP or CUDA, it's impor
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/65174
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/65174
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -574,6 +574,20 @@ benchmarks)
generate-cmake
check-cxx-benchmarks
;;
+check-lldb-data-formatters)
+clean
+${CMAKE}\
+-S "${MONOREPO_ROOT}/llvm" \
+-B "${BUILD_DIR}"
@@ -478,6 +480,43 @@ template class SourceLocScope final {
bool Enabled = false;
};
+template class StoredOpaqueValueScope final {
+public:
+ StoredOpaqueValueScope(ByteCodeExprGen *Ctx) : Ctx(Ctx) {}
+
+ bool VisitAndStoreOpaqueValue(const OpaqueValueExpr *Ove) {
+a
@@ -478,6 +480,43 @@ template class SourceLocScope final {
bool Enabled = false;
};
+template class StoredOpaqueValueScope final {
+public:
+ StoredOpaqueValueScope(ByteCodeExprGen *Ctx) : Ctx(Ctx) {}
+
+ bool VisitAndStoreOpaqueValue(const OpaqueValueExpr *Ove) {
+a
https://github.com/ZijunZhaoCCK updated
https://github.com/llvm/llvm-project/pull/65148
>From 02e9afd761228f401df4d9f8dfaaca44ffae0c6e Mon Sep 17 00:00:00 2001
From: zijunzhao
Date: Thu, 31 Aug 2023 20:08:32 +
Subject: [PATCH 01/12] [libc++] Implement ranges::contains
Differential Revision
https://github.com/ldionne updated
https://github.com/llvm/llvm-project/pull/65174
>From 9ff29c54e97dab71488ffe50ac871f007d357e4f Mon Sep 17 00:00:00 2001
From: Louis Dionne
Date: Fri, 9 Sep 2022 19:12:18 -0400
Subject: [PATCH 1/2] [libc++] Add a CI job for the LLDB data formatters
This patch
@@ -0,0 +1,190 @@
+//===--===//
+//
+// 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
MaskRay wrote:
> > The review context is on
> > [reviews.llvm.org/D145214](https://reviews.llvm.org/D145214) . Moving it
> > here just causes more confusion.
> > @alexfanqi is the author of the patch and they can drive a reland.
>
> Can you elaborate what is confusing? The patch was already re
https://github.com/hiraditya closed
https://github.com/llvm/llvm-project/pull/68735
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: AdityaK
Date: 2023-10-12T16:03:07-07:00
New Revision: 46cb8d9a325233ac11ed5e90367c43774294d87e
URL:
https://github.com/llvm/llvm-project/commit/46cb8d9a325233ac11ed5e90367c43774294d87e
DIFF:
https://github.com/llvm/llvm-project/commit/46cb8d9a325233ac11ed5e90367c43774294d87e.diff
LOG:
https://github.com/brad0 closed https://github.com/llvm/llvm-project/pull/67872
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Brad Smith
Date: 2023-10-12T18:05:49-04:00
New Revision: 8da1e3dd24a1cc6bc99bf3334009d2d19f21018f
URL:
https://github.com/llvm/llvm-project/commit/8da1e3dd24a1cc6bc99bf3334009d2d19f21018f
DIFF:
https://github.com/llvm/llvm-project/commit/8da1e3dd24a1cc6bc99bf3334009d2d19f21018f.diff
LO
brad0 wrote:
> Agreed: AFAICS the only open issue is whether the Solaris test should use the
> `GNU` label as I have done, introduce an new common one (like `UNIX`; there's
> nothing GNU-specific in that test), or really introduce a separate copy of
> the check under a different label per targ
https://github.com/rampitec closed
https://github.com/llvm/llvm-project/pull/68740
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/isuckatcs closed
https://github.com/llvm/llvm-project/pull/68246
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: isuckatcs
Date: 2023-10-12T23:26:44+02:00
New Revision: 403e0e8cd95f21d5f94f1e0663c2cfe48e54bf08
URL:
https://github.com/llvm/llvm-project/commit/403e0e8cd95f21d5f94f1e0663c2cfe48e54bf08
DIFF:
https://github.com/llvm/llvm-project/commit/403e0e8cd95f21d5f94f1e0663c2cfe48e54bf08.diff
LOG
isuckatcs wrote:
>it could come in handy in the future
For me it was helpful to check what is on the stack at the moment, so it's
definitely handy I think.
https://github.com/llvm/llvm-project/pull/68246
___
cfe-commits mailing list
cfe-commits@lists
https://github.com/ysyeda updated
https://github.com/llvm/llvm-project/pull/68926
>From 5ea8bea2bdf345e2baee07e0a94ac40bd0f109c4 Mon Sep 17 00:00:00 2001
From: Yusra Syeda
Date: Thu, 12 Oct 2023 16:56:27 -0400
Subject: [PATCH 1/2] This change adds support for the PPA2 section in zOS
---
clang
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 2ae3a712304870adf639a33547c1139a7b6304e5
5ea8bea2bdf345e2baee07e0a94ac40bd0f109c4 --
@@ -902,12 +890,16 @@ std::error_code
SampleProfileReaderExtBinaryBase::readFuncProfiles() {
for (const auto &NameOffset : FuncOffsetList) {
const auto &FContext = NameOffset.first;
auto FName = FContext.getName();
+StringRef FNameString;
+
@@ -902,12 +890,16 @@ std::error_code
SampleProfileReaderExtBinaryBase::readFuncProfiles() {
for (const auto &NameOffset : FuncOffsetList) {
const auto &FContext = NameOffset.first;
auto FName = FContext.getName();
+StringRef FNameString;
+
@@ -902,12 +890,16 @@ std::error_code
SampleProfileReaderExtBinaryBase::readFuncProfiles() {
for (const auto &NameOffset : FuncOffsetList) {
const auto &FContext = NameOffset.first;
auto FName = FContext.getName();
+StringRef FNameString;
+
@@ -0,0 +1,222 @@
+//===--- ProfileFuncRef.h - Sample profile function name ---*- 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: Apache-2
@@ -902,12 +890,16 @@ std::error_code
SampleProfileReaderExtBinaryBase::readFuncProfiles() {
for (const auto &NameOffset : FuncOffsetList) {
const auto &FContext = NameOffset.first;
auto FName = FContext.getName();
+StringRef FNameString;
@@ -0,0 +1,222 @@
+//===--- ProfileFuncRef.h - Sample profile function name ---*- 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: Apache-2
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Yusra Syeda (ysyeda)
Changes
This PR adds support for the PPA2 fields.
---
Patch is 21.58 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/68926.diff
12 Files Affected:
- (modified) cl
https://github.com/ysyeda created
https://github.com/llvm/llvm-project/pull/68926
This PR adds support for the PPA2 fields.
>From 5ea8bea2bdf345e2baee07e0a94ac40bd0f109c4 Mon Sep 17 00:00:00 2001
From: Yusra Syeda
Date: Thu, 12 Oct 2023 16:56:27 -0400
Subject: [PATCH] This change adds support
ymand wrote:
> Nice fix! So, it looks like our definition of "loop head" was wrong.
Indeed. We didn't think of the way control-flow constructs *inside* the
condition expression would be represented. Ironically (?), this is actually
simpler and fixes another issue besides, which is nice. ;)
ht
https://github.com/ymand updated https://github.com/llvm/llvm-project/pull/68923
>From 83486a35e6d0e754dd99369fc546d04afedbe923 Mon Sep 17 00:00:00 2001
From: Yitzhak Mandelbaum
Date: Thu, 12 Oct 2023 19:35:54 +
Subject: [PATCH 1/2] [clang][dataflow] Check for backedges directly (instead
of
https://github.com/rampitec updated
https://github.com/llvm/llvm-project/pull/68740
>From cc9e065a9218eb36750a2c2a4a4d08fae3f329fa Mon Sep 17 00:00:00 2001
From: Stanislav Mekhanoshin
Date: Wed, 4 Oct 2023 13:36:25 -0700
Subject: [PATCH 1/6] [AMDGPU] Change the representation of double literals
Author: Leonard Chan
Date: 2023-10-12T20:23:39Z
New Revision: ef388334ee5a3584255b9ef5b3fefdb244fa3fd7
URL:
https://github.com/llvm/llvm-project/commit/ef388334ee5a3584255b9ef5b3fefdb244fa3fd7
DIFF:
https://github.com/llvm/llvm-project/commit/ef388334ee5a3584255b9ef5b3fefdb244fa3fd7.diff
LOG:
brad0 wrote:
@MaskRay ping.
https://github.com/llvm/llvm-project/pull/67254
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Xazax-hun approved this pull request.
Nice fix! So, it looks like our definition of "loop head" was wrong.
https://github.com/llvm/llvm-project/pull/68923
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.or
ProExpertProg added a comment.
Is there an update on this? I agree adding a flag to control this behavior
would be good but the Google style guide clearly does not take the
`explicit(false)` option into consideration. Currently the only alternatives
are a disabling the check completely or addin
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 50ece4cba949787241b5fbfc94be6cfdc66e90ee
da8cacb47b8f80f7ecb1e86f98683be6c54b4e57 --
zygoloid wrote:
Does this work for function-scope operator declarations?
https://github.com/llvm/llvm-project/pull/68922
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/usx95 updated https://github.com/llvm/llvm-project/pull/68922
>From 0c71b6bdd557ff4b9ad9a4ec4f0919e3460596b0 Mon Sep 17 00:00:00 2001
From: Utkarsh Saxena
Date: Thu, 12 Oct 2023 21:35:52 +0200
Subject: [PATCH 1/2] Find opertor!= in the correct namespace
---
clang/lib/Sema/Se
https://github.com/ymand updated https://github.com/llvm/llvm-project/pull/68923
>From 83486a35e6d0e754dd99369fc546d04afedbe923 Mon Sep 17 00:00:00 2001
From: Yitzhak Mandelbaum
Date: Thu, 12 Oct 2023 19:35:54 +
Subject: [PATCH] [clang][dataflow] Check for backedges directly (instead of
loo
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Utkarsh Saxena (usx95)
Changes
`S.getScopeForContext` determins the **active** scope associated with the given
`declContext`.
This fails to find the matching `operator!=` if candidate `operator==` was
found via ADL since that scope is no
https://github.com/usx95 ready_for_review
https://github.com/llvm/llvm-project/pull/68922
___
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
Author: Yitzhak Mandelbaum (ymand)
Changes
Widen on backedge nodes, instead of nodes with a loop statement as terminator.
This fixes #67834 and a precision loss from assignment in a loop
condition. The
commit contains tests for both of these issue
https://github.com/ymand created https://github.com/llvm/llvm-project/pull/68923
Widen on backedge nodes, instead of nodes with a loop statement as terminator.
This fixes #67834 and a precision loss from assignment in a loop condition. The
commit contains tests for both of these issues.
>From d
https://github.com/usx95 created https://github.com/llvm/llvm-project/pull/68922
`S.getScopeForContext` determins the **active** scope associated with the given
`declContext`.
This fails to find the matching `operator!=` if candidate `operator==` was
found via ADL since that scope is not activ
1 - 100 of 259 matches
Mail list logo
