https://github.com/ilovepi edited
https://github.com/llvm/llvm-project/pull/141699
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman approved this pull request.
LGTM! Failing docs build in precommit CI appears to be unrelated
https://github.com/llvm/llvm-project/pull/139429
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org
chandraghale wrote:
@alexey-bataev Added few more changes on top, to be on accordance with spec.
Shared copy is updated with values from private copies until all updates are
complete, before combining into the original list item. Hope the changes are
fine with you.
https://github.com/llvm/ll
Author: Andy Kaylor
Date: 2025-05-30T10:26:28-07:00
New Revision: 294a182021c6a9f93e1eebd173abb7fb6d840fc5
URL:
https://github.com/llvm/llvm-project/commit/294a182021c6a9f93e1eebd173abb7fb6d840fc5
DIFF:
https://github.com/llvm/llvm-project/commit/294a182021c6a9f93e1eebd173abb7fb6d840fc5.diff
L
https://github.com/andykaylor closed
https://github.com/llvm/llvm-project/pull/142059
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AmrDeveloper closed
https://github.com/llvm/llvm-project/pull/141411
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
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 HEAD~1 HEAD --extensions h,cpp --
clang-tools-extra/clangd/Hover.cpp clang-tools-extr
HighCommander4 wrote:
Carrying over review request to @kadircet from the original combined patch at
https://github.com/llvm/llvm-project/pull/128591#issuecomment-2782334206:
> @kadircet, you were involved with earlier rounds of review on the old patch
> (https://reviews.llvm.org/D143112) and d
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-x86_64-debian`
running on `lldb-x86_64-debian` while building `clang` at step 6 "test".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/162/builds/23552
Here is the relevant piece of the build lo
https://github.com/qinkunbao closed
https://github.com/llvm/llvm-project/pull/142168
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman created
https://github.com/llvm/llvm-project/pull/142195
Microsoft helpfully defines `THIS` to `void` in two different platform SDK
headers, at least one of which is reachable via . We have a user who
ran into a build because of `THIS` unfortunate macro name col
llvmbot wrote:
@llvm/pr-subscribers-platform-windows
Author: Aaron Ballman (AaronBallman)
Changes
Microsoft helpfully defines `THIS` to `void` in two different platform SDK
headers, at least one of which is reachable via . We have a
user who ran into a build because of `THIS`
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Aaron Ballman (AaronBallman)
Changes
Microsoft helpfully defines `THIS` to `void` in two different platform SDK
headers, at least one of which is reachable via . We have a
user who ran into a build because of `THIS` unfortunate
https://github.com/erichkeane approved this pull request.
Sure they don't have a `#define This ` somewhere :) ?
https://github.com/llvm/llvm-project/pull/142195
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
https://github.com/AmrDeveloper updated
https://github.com/llvm/llvm-project/pull/142012
>From f1854921327247d4449fcac6375a3b43b8e69996 Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Thu, 29 May 2025 20:58:56 +0200
Subject: [PATCH] [CIR] Upstream ConvertVectorExpr
---
clang/lib/CIR/CodeGen
@@ -1245,30 +1245,23 @@ void CodeGenFunction::EmitBoundsCheckImpl(const Expr
*E, llvm::Value *Bound,
}
llvm::DILocation *CodeGenFunction::SanitizerAnnotateDebugInfo(
-SanitizerKind::SanitizerOrdinal CheckKindOrdinal) {
- std::string Label;
- switch (CheckKindOrdinal) {
https://github.com/xlauko approved this pull request.
lgtm
https://github.com/llvm/llvm-project/pull/142012
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin approved this pull request.
https://github.com/llvm/llvm-project/pull/141470
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman approved this pull request.
This looks reasonable to me, but please wait for @erichkeane to sign off before
landing
https://github.com/llvm/llvm-project/pull/137882
___
cfe-commits mailing list
cfe-commits@lists.llvm.or
https://github.com/qinkunbao edited
https://github.com/llvm/llvm-project/pull/142074
___
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-static-analyzer-1
Author: Donát Nagy (NagyDonat)
Changes
By design the `Location` data member of a `CheckerContext` is always a
`ProgramPoint` which is tagged with the currently active checker (note that all
checker classes are subclasses of `Pro
tbaederr wrote:
Ah, sorry about that. Thanks.
https://github.com/llvm/llvm-project/pull/141952
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/NagyDonat edited
https://github.com/llvm/llvm-project/pull/142132
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/NagyDonat edited
https://github.com/llvm/llvm-project/pull/142132
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tbaederr edited
https://github.com/llvm/llvm-project/pull/141470
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
philnik777 wrote:
Gentle ping @AaronBallman @cor3ntin
https://github.com/llvm/llvm-project/pull/116709
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/NagyDonat created
https://github.com/llvm/llvm-project/pull/142132
By design the `Location` data member of a `CheckerContext` is always a
`ProgramPoint` which is tagged with the currently active checker (note that all
checker classes are subclasses of `ProgramPointTag`). Thi
tbaederr wrote:
> Can you elaborate in your summary.
>
> I am guessing because the other checks are a lot cheaper and this overall
> improves performance?
Yep. Updated the description.
https://github.com/llvm/llvm-project/pull/141470
___
cfe-commits
https://github.com/NagyDonat ready_for_review
https://github.com/llvm/llvm-project/pull/142132
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: FYK
Date: 2025-05-30T08:13:53-06:00
New Revision: d27a210a77af63568db9f829702b4b2c98473a46
URL:
https://github.com/llvm/llvm-project/commit/d27a210a77af63568db9f829702b4b2c98473a46
DIFF:
https://github.com/llvm/llvm-project/commit/d27a210a77af63568db9f829702b4b2c98473a46.diff
LOG: Add
llvmbot wrote:
@llvm/pr-subscribers-backend-amdgpu
Author: Nikita Popov (nikic)
Changes
The InstrProf headers are very expensive. Avoid including them in all of
CodeGen/ by making the CodeGenPGO member behind a unqiue_ptr.
This reduces clang build time by 0.8%:
https://llvm-compile-time-
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Nikita Popov (nikic)
Changes
The InstrProf headers are very expensive. Avoid including them in all of
CodeGen/ by making the CodeGenPGO member behind a unqiue_ptr.
This reduces clang build time by 0.8%:
https://llvm-compile-time-t
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Nikita Popov (nikic)
Changes
The InstrProf headers are very expensive. Avoid including them in all of
CodeGen/ by making the CodeGenPGO member behind a unqiue_ptr.
This reduces clang build time by 0.8%:
https://llvm-compile-time-tracker.c
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`amdgpu-offload-rhel-9-cmake-build-only` running on `rocm-docker-rhel-9` while
building `clang,flang,llvm` at step 4 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/205/builds/10879
Here i
https://github.com/nikic created
https://github.com/llvm/llvm-project/pull/142155
The InstrProf headers are very expensive. Avoid including them in all of
CodeGen/ by making the CodeGenPGO member behind a unqiue_ptr.
This reduces clang build time by 0.8%:
https://llvm-compile-time-tracker.com/
https://github.com/tarunprabhu closed
https://github.com/llvm/llvm-project/pull/136098
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -764,7 +764,8 @@ static bool interp__builtin_addressof(InterpState &S,
CodePtr OpPC,
const InterpFrame *Frame,
const CallExpr *Call) {
assert(Call->getArg(0)->isLValue());
- PrimType PtrT = S.get
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`openmp-offload-amdgpu-runtime-2` running on `rocm-worker-hw-02` while building
`clang,flang,llvm` at step 5 "compile-openmp".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/10/builds/6370
Here is th
https://github.com/DKLoehr created
https://github.com/llvm/llvm-project/pull/142158
This improves the documentation for the unique-object-duplication warning. It
clarifies the conditions for the warning to fire, provides clearer instructions
on how to resolve it, and adjusts wording to be more
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Tarun Prabhu (tarunprabhu)
Changes
Reverts llvm/llvm-project#136098
---
Patch is 29.10 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/142159.diff
21 Files Affected:
- (modified) clang
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Devon Loehr (DKLoehr)
Changes
This improves the documentation for the unique-object-duplication warning. It
clarifies the conditions for the warning to fire, provides clearer instructions
on how to resolve it, and adjusts wording to be mo
https://github.com/jplehr approved this pull request.
LG
https://github.com/llvm/llvm-project/pull/142159
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Tarun Prabhu
Date: 2025-05-30T08:27:08-06:00
New Revision: 597340b5b666bdee2887f56c111407b6737cbf34
URL:
https://github.com/llvm/llvm-project/commit/597340b5b666bdee2887f56c111407b6737cbf34
DIFF:
https://github.com/llvm/llvm-project/commit/597340b5b666bdee2887f56c111407b6737cbf34.diff
https://github.com/tarunprabhu closed
https://github.com/llvm/llvm-project/pull/142159
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`amdgpu-offload-rhel-8-cmake-build-only` running on `rocm-docker-rhel-8` while
building `clang,flang,llvm` at step 4 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/204/builds/10902
Here i
tarunprabhu wrote:
@fanju110
The PR caused some buildbot failures, so I have
[reverted](https://github.com/llvm/llvm-project/commit/597340b5b666bdee2887f56c111407b6737cbf34)
it. You can see the error
[here](https://lab.llvm.org/buildbot/#/builders/203/builds/12089/steps/7/logs/stdio).
htt
@@ -0,0 +1,17 @@
+! RUN: env FCC_OVERRIDE_OPTIONS="#+-Os +-Oz +-O +-O3 +-Oignore +a +b +c xb Xa
Omagic ^-### " %flang -target x86_64-unknown-linux-gnu %s -O2 b -O3 2>&1 |
FileCheck %s
tarunprabhu wrote:
Should this also be `--target`?
```suggestion
! RUN: env
https://github.com/steakhal approved this pull request.
LGTM, thanks.
https://github.com/llvm/llvm-project/pull/142132
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-flang-driver
@llvm/pr-subscribers-clang-codegen
Author: Tarun Prabhu (tarunprabhu)
Changes
Reverts llvm/llvm-project#136098
---
Patch is 29.10 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/142159.diff
21
https://github.com/tarunprabhu created
https://github.com/llvm/llvm-project/pull/142159
Reverts llvm/llvm-project#136098
>From b9503fe262c416111ee77be30767a791cf750fb8 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Fri, 30 May 2025 08:22:15 -0600
Subject: [PATCH] =?UTF-8?q?Revert=20"Add=20I
https://github.com/tarunprabhu closed
https://github.com/llvm/llvm-project/pull/142073
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `ppc64le-flang-rhel-clang`
running on `ppc64le-flang-rhel-test` while building `clang,flang,mlir` at step
6 "test-build-unified-tree-check-flang".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/157/bui
https://github.com/tarunprabhu edited
https://github.com/llvm/llvm-project/pull/140556
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clangir
Author: Andy Kaylor (andykaylor)
Changes
This enables us to compile C++ code with nested structures. The necessary
support for this was already in place, but we were hitting an NYI error in a
place where the implementation only needed to check
https://github.com/andykaylor created
https://github.com/llvm/llvm-project/pull/142205
This enables us to compile C++ code with nested structures. The necessary
support for this was already in place, but we were hitting an NYI error in a
place where the implementation only needed to check for
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Andy Kaylor (andykaylor)
Changes
This enables us to compile C++ code with nested structures. The necessary
support for this was already in place, but we were hitting an NYI error in a
place where the implementation only needed to check fo
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/142164
>From 662e07aa9bb6560f37c079ba6f13be17e7885b48 Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Fri, 30 May 2025 15:44:09 +0100
Subject: [PATCH 1/5] [clang][Frontend] Add overload to ASTPrinter that doesn't
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/142164
>From 662e07aa9bb6560f37c079ba6f13be17e7885b48 Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Fri, 30 May 2025 15:44:09 +0100
Subject: [PATCH 1/4] [clang][Frontend] Add overload to ASTPrinter that doesn't
https://github.com/tcottin updated
https://github.com/llvm/llvm-project/pull/140498
>From 8fadd8d51fa3d96c7fb82b9d749ef3f35441ac64 Mon Sep 17 00:00:00 2001
From: Tim Cottin
Date: Mon, 19 May 2025 06:26:36 +
Subject: [PATCH 1/2] [clangd] Improve Markup Rendering
---
clang-tools-extra/clang
Author: Qinkun Bao
Date: 2025-05-30T12:01:26-04:00
New Revision: 6a47241c9983c46d805034821f04c34a475a254f
URL:
https://github.com/llvm/llvm-project/commit/6a47241c9983c46d805034821f04c34a475a254f
DIFF:
https://github.com/llvm/llvm-project/commit/6a47241c9983c46d805034821f04c34a475a254f.diff
LO
@@ -972,6 +972,12 @@ class Sema final : public SemaBase {
/// Calls \c Lexer::getLocForEndOfToken()
SourceLocation getLocForEndOfToken(SourceLocation Loc, unsigned Offset = 0);
+ /// Calls \c Lexer::findNextToken() to find the next token, and if the
+ /// locations of bo
https://github.com/andykaylor updated
https://github.com/llvm/llvm-project/pull/141973
>From 70aa8d668cf610eb7d2486b60112591a19d3db89 Mon Sep 17 00:00:00 2001
From: Andy Kaylor
Date: Fri, 23 May 2025 11:03:05 -0700
Subject: [PATCH 1/2] [CIR] Add support for global linkage and visibility
This c
https://github.com/fmayer requested changes to this pull request.
How is this going to be useful? We should make some effort to try to determine
where the overhead comes from. I don't like that the specific array_bounds
inline function is going away _even though we have a specific codepath for
nikic wrote:
For the record, the manual backport PR is at:
https://github.com/llvm/llvm-project/pull/141957
https://github.com/llvm/llvm-project/pull/141890
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman
rockwotj wrote:
Based on #141114 and #141858 should this PR be reverted?
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
Sisyph wrote:
> Is there a test that needs to be added here?
With just the change to BuiltinsAMDGPU.def, we will get errors in
Clang :: CodeGenOpenCL/builtins-amdgcn-gfx950.cl
Clang :: CodeGenOpenCL/builtins-amdgcn-load-to-lds.cl
Clang :: SemaOpenCL/builtins-amdgcn-load-to-lds-err.cl
wh
YutongZhuu wrote:
> LGTM! Failing docs build in precommit CI appears to be unrelated
Hi, I don't have commit access, so I might need you to merge it for me :)
https://github.com/llvm/llvm-project/pull/139429
___
cfe-commits mailing list
cfe-commits@li
https://github.com/AmrDeveloper updated
https://github.com/llvm/llvm-project/pull/123806
>From 38031e9aed519f4fd3b21dff7e79c69a0150efd9 Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Tue, 21 Jan 2025 20:11:21 +0100
Subject: [PATCH] [Clang] Improve testing of non std functions with name
infi
https://github.com/sarnex edited
https://github.com/llvm/llvm-project/pull/142019
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -223,21 +272,114 @@ void CIRRecordLowering::fillOutputFields() {
fieldTypes.size() - 1;
// A field without storage must be a bitfield.
assert(!cir::MissingFeatures::bitfields());
+ if (!member.data)
+setBitFieldInfo(member.fieldDecl, memb
https://github.com/ilovepi approved this pull request.
LGTM. We may want to tweak the template some more, but we should get the test
in place now and follow up if we want to change the aesthetics later.
https://github.com/llvm/llvm-project/pull/142045
___
https://github.com/nabijaczleweli created
https://github.com/llvm/llvm-project/pull/142213
You can absolutely push from a shallow clone, this is just false
From 0827ae2057b4a05d797587e09f6f198c01abe00f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=D0=BD=D0=B0=D0=B1?=
Date: Fri, 30 May 2025 22:50:5
github-actions[bot] wrote:
Thank you for submitting a Pull Request (PR) to the LLVM Project!
This PR will be automatically labeled and the relevant teams will be notified.
If you wish to, you can add reviewers by using the "Reviewers" section on this
page.
If this is not working for you, it
https://github.com/sarnex edited
https://github.com/llvm/llvm-project/pull/142019
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/usx95 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/142195
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sarnex edited
https://github.com/llvm/llvm-project/pull/142019
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sarnex edited
https://github.com/llvm/llvm-project/pull/142019
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sarnex edited
https://github.com/llvm/llvm-project/pull/142019
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -223,21 +272,114 @@ void CIRRecordLowering::fillOutputFields() {
fieldTypes.size() - 1;
// A field without storage must be a bitfield.
assert(!cir::MissingFeatures::bitfields());
+ if (!member.data)
+setBitFieldInfo(member.fieldDecl, memb
@@ -223,21 +272,114 @@ void CIRRecordLowering::fillOutputFields() {
fieldTypes.size() - 1;
// A field without storage must be a bitfield.
assert(!cir::MissingFeatures::bitfields());
+ if (!member.data)
+setBitFieldInfo(member.fieldDecl, memb
@@ -14,6 +14,105 @@
namespace clang::CIRGen {
+/// Record with information about how a bitfield should be accessed. This is
+/// very similar to what LLVM codegen does, once CIR evolves it's possible we
+/// can use a more higher level representation.
+///
+/// Often we lay o
@@ -0,0 +1,79 @@
+// RUN: %clang_cc1 -std=c++17 -triple x86_64-unknown-linux-gnu -fclangir
-emit-cir %s -o %t.cir
+// RUN: FileCheck --input-file=%t.cir %s --check-prefix=CIR
+// RUN: %clang_cc1 -std=c++17 -triple x86_64-unknown-linux-gnu -fclangir
-emit-llvm %s -o %t-cir.ll
+//
@@ -14,6 +14,105 @@
namespace clang::CIRGen {
+/// Record with information about how a bitfield should be accessed. This is
+/// very similar to what LLVM codegen does, once CIR evolves it's possible we
+/// can use a more higher level representation.
+///
+/// Often we lay o
@@ -223,21 +272,114 @@ void CIRRecordLowering::fillOutputFields() {
fieldTypes.size() - 1;
// A field without storage must be a bitfield.
assert(!cir::MissingFeatures::bitfields());
+ if (!member.data)
+setBitFieldInfo(member.fieldDecl, memb
@@ -223,21 +272,114 @@ void CIRRecordLowering::fillOutputFields() {
fieldTypes.size() - 1;
// A field without storage must be a bitfield.
assert(!cir::MissingFeatures::bitfields());
+ if (!member.data)
+setBitFieldInfo(member.fieldDecl, memb
@@ -223,21 +272,114 @@ void CIRRecordLowering::fillOutputFields() {
fieldTypes.size() - 1;
// A field without storage must be a bitfield.
assert(!cir::MissingFeatures::bitfields());
+ if (!member.data)
+setBitFieldInfo(member.fieldDecl, memb
@@ -223,21 +272,114 @@ void CIRRecordLowering::fillOutputFields() {
fieldTypes.size() - 1;
// A field without storage must be a bitfield.
assert(!cir::MissingFeatures::bitfields());
+ if (!member.data)
+setBitFieldInfo(member.fieldDecl, memb
@@ -223,21 +272,114 @@ void CIRRecordLowering::fillOutputFields() {
fieldTypes.size() - 1;
// A field without storage must be a bitfield.
assert(!cir::MissingFeatures::bitfields());
+ if (!member.data)
+setBitFieldInfo(member.fieldDecl, memb
@@ -172,6 +195,32 @@ CIRRecordLowering::CIRRecordLowering(CIRGenTypes
&cirGenTypes,
zeroInitializable(true), zeroInitializableAsBase(true), packed(packed),
padded(false) {}
+void CIRRecordLowering::setBitFieldInfo(const FieldDecl *fd,
+
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`openmp-offload-amdgpu-runtime-2` running on `rocm-worker-hw-02` while building
`clang` at step 6 "test-openmp".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/10/builds/6398
Here is the relevant pie
https://github.com/nabijaczleweli edited
https://github.com/llvm/llvm-project/pull/142213
___
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: наб (nabijaczleweli)
Changes
You can absolutely push from a shallow clone, this is just false
---
Full diff: https://github.com/llvm/llvm-project/pull/142213.diff
1 Files Affected:
- (modified) clang/www/get_started.html (+1-1)
`
zygoloid wrote:
Ping x8
@PiotrZSL Any chance you'll find time to look at this soon, or should I look
for other reviewers?
https://github.com/llvm/llvm-project/pull/130369
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/
https://github.com/thurstond converted_to_draft
https://github.com/llvm/llvm-project/pull/141997
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
andykaylor wrote:
@bcardosolopes, I have some general concerns about the way that we're
representing bitfields and bitfield accesses. It seems fundamentally
inconsistent with the way we represent unions. For example:
```
union U {
char a;
int b;
long c;
};
```
becomes
`!cir.record ->
!c
https://github.com/sarnex edited
https://github.com/llvm/llvm-project/pull/142019
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sisyph updated
https://github.com/llvm/llvm-project/pull/138141
>From f5cdefe8200d9c9f567d6b4a276a5587e44ac1fa Mon Sep 17 00:00:00 2001
From: Joe Nash
Date: Thu, 1 May 2025 10:00:47 -0400
Subject: [PATCH] [Sema] Fix bug in builtin AS override
Fix the logic in rewriteBuiltinF
https://github.com/DeanSturtevant1 created
https://github.com/llvm/llvm-project/pull/142189
A previous to InterpBuiltin.cpp fixed an unused variable warning by using
[[maybe unused]] and (void).
The code actually serves no useful purpose in non-debug builds, so let's not
include it there.
>F
Sisyph wrote:
Rebased, PTAL
https://github.com/llvm/llvm-project/pull/138141
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
krzysz00 wrote:
Ok, so this is a pure bugfix
Since this bit of infrastructure's kinda underdocumented, why did the `"t"`
flag need to be removed and what does it mean?
https://github.com/llvm/llvm-project/pull/138141
___
cfe-commits mailing list
cfe-
201 - 300 of 384 matches
Mail list logo
