https://github.com/dhoekwater closed
https://github.com/llvm/llvm-project/pull/82662
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1306,15 +1306,73 @@ float min(float __x, float __y) { return
__builtin_fminf(__x, __y); }
__DEVICE__
double min(double __x, double __y) { return __builtin_fmin(__x, __y); }
-#if !defined(__HIPCC_RTC__) && !defined(__OPENMP_AMDGCN__)
-__host__ inline static int min(int __a
https://github.com/ymand approved this pull request.
https://github.com/llvm/llvm-project/pull/82986
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ymand edited https://github.com/llvm/llvm-project/pull/82986
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2392,14 +2392,88 @@ TEST(TransferTest, InitListExprAsUnion) {
} F;
public:
- constexpr target() : F{nullptr} {}
+ constexpr target() : F{nullptr} {
+int *null = nullptr;
+F.b; // Make sure we reference 'b' so it is modeled.
+//
@@ -2392,14 +2392,88 @@ TEST(TransferTest, InitListExprAsUnion) {
} F;
public:
- constexpr target() : F{nullptr} {}
+ constexpr target() : F{nullptr} {
+int *null = nullptr;
+F.b; // Make sure we reference 'b' so it is modeled.
+//
https://github.com/brad0 created https://github.com/llvm/llvm-project/pull/83216
…D and Haiku
Based on https://github.com/llvm/llvm-project/pull/76432
>From e9dca00cb002d223b4148d648f4f31b4cfc31d63 Mon Sep 17 00:00:00 2001
From: Brad Smith
Date: Tue, 27 Feb 2024 21:07:09 -0500
Subject: [PATCH]
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Brad Smith (brad0)
Changes
…D and Haiku
Based on https://github.com/llvm/llvm-project/pull/76432
---
Full diff: https://github.com/llvm/llvm-project/pull/83216.diff
4 Files Affected:
- (modified) clang/lib/Driver/ToolChains/Free
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 bfcf7a0707592ccc7fd9e805aeb36c4da3f315a6
e9dca00cb002d223b4148d648f4f31b4cfc31d63 --
https://github.com/brad0 updated https://github.com/llvm/llvm-project/pull/83216
>From 4409a37ff160edb3c41c7650c181a5c9a60c397b Mon Sep 17 00:00:00 2001
From: Brad Smith
Date: Tue, 27 Feb 2024 21:07:09 -0500
Subject: [PATCH] [Driver][RISCV] Forward --no-relax option to linker for
RISC-V on *BSD
https://github.com/cyndyishida updated
https://github.com/llvm/llvm-project/pull/82552
>From b0cc1eeba893c24d5358cb49189da2e287daf972 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Wed, 21 Feb 2024 14:56:02 -0800
Subject: [PATCH 1/2] [InstallAPI] Hookup Input files & basic ASTVisitor
This p
@@ -0,0 +1,165 @@
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
UTC_ARGS: --version 4
+; RUN: llc < %s -mtriple=i686-unknown-unknown --relocation-model=pic
-enable-tlsdesc | FileCheck %s --check-prefix=X86
MaskRay wrote:
`-unkno
@@ -18515,17 +18515,17 @@ X86TargetLowering::LowerGlobalAddress(SDValue Op,
SelectionDAG &DAG) const {
return LowerGlobalOrExternal(Op, DAG, /*ForCall=*/false);
}
-static SDValue
-GetTLSADDR(SelectionDAG &DAG, SDValue Chain, GlobalAddressSDNode *GA,
- SDValue *InG
@@ -0,0 +1,165 @@
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
UTC_ARGS: --version 4
+; RUN: llc < %s -mtriple=i686-unknown-unknown --relocation-model=pic
-enable-tlsdesc | FileCheck %s --check-prefix=X86
+; RUN: llc < %s -mtriple=x86_64-pc-linu
https://github.com/wangpc-pp approved this pull request.
LGTM.
Will it be in LLVM 18? Or we need more time to examine its robustness?
https://github.com/llvm/llvm-project/pull/83195
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.
topperc wrote:
> LGTM.
>
> Will it be in LLVM 18? Or we need more time to examine its robustness?
The bar for LLVM 18 is pretty high at this point. I doubt this qualifies.
https://github.com/llvm/llvm-project/pull/83195
___
cfe-commits mailing list
c
haoNoQ wrote:
Hi! I wonder if path-sensitive analysis is useful here at all. If you simply
warn every time you see arithmetic performed on a pointer to a class with at
least one virtual method, wouldn't that be quite accurate most of the time?
At a glance, such operation only makes sense when
haoNoQ wrote:
Yes I like this perspective: "Schrödinger's need-for-pre-commit-approval" isn't
a great way to communicate 😅 I'll be more clear in the future.
https://github.com/llvm/llvm-project/pull/80371
___
cfe-commits mailing list
cfe-commits@lists
https://github.com/torshepherd updated
https://github.com/llvm/llvm-project/pull/79867
>From 94dee94becb7d79b087e183754602e08a5c4669d Mon Sep 17 00:00:00 2001
From: Tor Shepherd
Date: Mon, 29 Jan 2024 11:44:25 -0500
Subject: [PATCH] [clangd] Add fix-all CodeActions
---
clang-tools-extra/clang
dwblaikie wrote:
Some test coverage would be good to help demonstrate the issue/fix
https://github.com/llvm/llvm-project/pull/83175
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
wsmoses wrote:
Hm actually reopening, for some reason the metadata isn't being pushed at O0.
https://github.com/llvm/llvm-project/pull/83059
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com
https://github.com/wsmoses reopened
https://github.com/llvm/llvm-project/pull/83059
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
shafik wrote:
Can you explain which cases the `if (!ArraySize)` condition was catching before
and why the change does not effect those cases? My fear is that we are breaking
other cases but we don't have test coverage for those cases and we are missing
those breaks.
https://github.com/llvm/ll
mahtohappy wrote:
@shafik I'm not aware of which cases it was catching before, I was hoping
testing will reveal those testcases. I agree with your thinking that we might
be breaking something and and don't have test coverage for it. I'll leave this
condition as it is and add a separate check f
https://github.com/sandeepkosuri created
https://github.com/llvm/llvm-project/pull/83223
- adds Parse and Sema support for the `declare target` directive inside a
function scope.
>From cbf1b4409e379309ae3d942b3dbec0964b9ee0d1 Mon Sep 17 00:00:00 2001
From: Sandeep Kosuri
Date: Tue, 27 Feb 202
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Sandeep Kosuri (sandeepkosuri)
Changes
- adds Parse and Sema support for the `declare target` directive inside a
function scope.
---
Full diff: https://github.com/llvm/llvm-project/pull/83223.diff
5 Files Affected:
- (modified) clang/i
https://github.com/cyndyishida updated
https://github.com/llvm/llvm-project/pull/82552
>From a7d0c1cf15764ee2ccbeb609fbaf014fdbde81b1 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Wed, 21 Feb 2024 14:56:02 -0800
Subject: [PATCH] [InstallAPI] Hookup Input files & basic ASTVisitor
This patch
t-tye wrote:
I am not clear why new functions need to be added for this, as I think there
are existing functions that already do this.
https://github.com/llvm/llvm-project/pull/79236
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://list
https://github.com/arsenm commented:
Title should be rephrased; this doesn't have anything to do with inlining
https://github.com/llvm/llvm-project/pull/81058
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailm
https://github.com/minglotus-6 edited
https://github.com/llvm/llvm-project/pull/66825
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
arsenm wrote:
> So overall, the practical effect of the `denormal-fp-math` attribute being
> set incorrectly doesn't appear to matter.
It matters more for AMDGPU, where we need to care because some instructions
just don't respect denormals. We legalize some operations differently depending
on
Author: Ryosuke Niwa
Date: 2024-02-27T22:25:55-08:00
New Revision: 4d04a40adb68f284350831911a658715134c66d8
URL:
https://github.com/llvm/llvm-project/commit/4d04a40adb68f284350831911a658715134c66d8
DIFF:
https://github.com/llvm/llvm-project/commit/4d04a40adb68f284350831911a658715134c66d8.diff
https://github.com/rniwa closed https://github.com/llvm/llvm-project/pull/82291
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mahtohappy updated
https://github.com/llvm/llvm-project/pull/83124
>From 7f6bbd2a9e5ab162b0efb5c6dcca91e1d83a0bce Mon Sep 17 00:00:00 2001
From: mahtohappy
Date: Tue, 27 Feb 2024 03:13:51 -0800
Subject: [PATCH] [Clang][Sema] placement new initializes typedef array with
corre
https://github.com/minglotus-6 ready_for_review
https://github.com/llvm/llvm-project/pull/66825
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
minglotus-6 wrote:
New changes since last review besides clang-formats
1. Merged profile format change from main branch.
2. In instrumentation pass, emit unsupported warnings for non-ELF object file
formats.
- Updated vtable_profile.ll and add vtable_prof_unsupported.ll as test
cases.
htt
kpdev wrote:
@AaronBallman @kongy Hi! Could you please take a look at this patch?
https://github.com/llvm/llvm-project/pull/78253
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ChuanqiXu9 wrote:
@mordante if we want this for 18, we need to land and backport it in this week.
https://github.com/llvm/llvm-project/pull/82160
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cf
301 - 338 of 338 matches
Mail list logo