llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `llvm-clang-x86_64-sie-win`
running on `sie-win-worker` while building `clang` at step 7
"test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/46/builds/13857
Here is th
https://github.com/erichkeane edited
https://github.com/llvm/llvm-project/pull/131945
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/dkolsen-pgi closed
https://github.com/llvm/llvm-project/pull/131945
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
andykaylor wrote:
The `CIR/Lowering/func-simple.cpp` test is failing after this merge because of
my change to introduce the canonicalize pass. I'll update the test as part of
my type alias patch.
https://github.com/llvm/llvm-project/pull/131945
___
c
https://github.com/dkolsen-pgi updated
https://github.com/llvm/llvm-project/pull/131945
>From ef54ceca65c8a62544651cbbd30967efc7adec36 Mon Sep 17 00:00:00 2001
From: David Olsen
Date: Tue, 18 Mar 2025 17:37:26 -0700
Subject: [PATCH 1/2] [CIR] Upstream a basic version of class LexicalScope
Upst
https://github.com/erichkeane approved this pull request.
Want those nits from my last patch fixed, but forgot to click the 'approve'
button.
https://github.com/llvm/llvm-project/pull/131945
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http
@@ -149,14 +156,125 @@ void CIRGenFunction::declare(mlir::Value addrVal, const
Decl *var, QualType ty,
allocaOp.setConstantAttr(mlir::UnitAttr::get(&getMLIRContext()));
}
+void CIRGenFunction::LexicalScope::cleanup() {
+ CIRGenBuilderTy &builder = cgf.builder;
+ Lexical
@@ -149,14 +156,125 @@ void CIRGenFunction::declare(mlir::Value addrVal, const
Decl *var, QualType ty,
allocaOp.setConstantAttr(mlir::UnitAttr::get(&getMLIRContext()));
}
+void CIRGenFunction::LexicalScope::cleanup() {
+ CIRGenBuilderTy &builder = cgf.builder;
+ Lexical
https://github.com/erichkeane commented:
2 nits, else LGTM.
https://github.com/llvm/llvm-project/pull/131945
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
dkolsen-pgi wrote:
There are some small functions in this change that are not called or otherwise
used in this change. But they will definitely be used in future upstream
patches. Since they are small and don't bring in any new dependencies and are
complete (won't need to be changed in futur
llvmbot wrote:
@llvm/pr-subscribers-clangir
Author: David Olsen (dkolsen-pgi)
Changes
Upstream the parts of class `CIRGenFunction::LexicalScope` that implement
function return values. There is a bit of other functionality in here, such as
the implicit `cir.yield` at the end of a non-fun
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: David Olsen (dkolsen-pgi)
Changes
Upstream the parts of class `CIRGenFunction::LexicalScope` that implement
function return values. There is a bit of other functionality in here, such as
the implicit `cir.yield` at the end of a non-funct
https://github.com/dkolsen-pgi created
https://github.com/llvm/llvm-project/pull/131945
Upstream the parts of class `CIRGenFunction::LexicalScope` that implement
function return values. There is a bit of other functionality in here, such as
the implicit `cir.yield` at the end of a non-functio
13 matches
Mail list logo
