glandium wrote:
> Which used to be MT and with this change is now MD.
(because our script was still using LLVM_USE_CRT_RELEASE)
https://github.com/llvm/llvm-project/pull/66850
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
glandium wrote:
This unveiled an interesting problem in our builds: `llvm-config --cxxflags`
doesn't include the `/MT`/`/MD` flag, so when building something that links
with one of the .lib from LLVM, you can end up with a failure because you're
not building with the same flag as LLVM was
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 a06e94cf3b375cdfd81f42f60e16ce0c3783ff4a
d987a8523086ab8b6d49f395429afda75769a731 --
rajkumarananthu wrote:
Hi @AaronBallman I could see a revert of my change here:
https://reviews.llvm.org/rGa6acf3fd49a20c570a390af2a3c84e10b9545b68
Can you please help me confirm whether the change is accepted or reverted.
Should we resubmit the changes here?
Thanks
Rajkumar Ananthu
llvmbot wrote:
@llvm/pr-subscribers-clang
Changes
While working on #68377 inspecting `Allocate()` calls, I found out that
there are couple of places where we forget to use placement-new to create
objects in the allocated memory.
---
Full diff:
https://github.com/Endilll updated
https://github.com/llvm/llvm-project/pull/68382
>From d987a8523086ab8b6d49f395429afda75769a731 Mon Sep 17 00:00:00 2001
From: Vlad Serebrennikov
Date: Fri, 6 Oct 2023 07:20:07 +0300
Subject: [PATCH 1/2] [clang][NFC] Add missing placement-new after Allocate()
https://github.com/Endilll created
https://github.com/llvm/llvm-project/pull/68382
While working on #68377 inspecting `Allocate()` calls, I found out that there
are couple of places where we forget to use placement-new to create objects in
the allocated memory.
>From
llvmbot wrote:
@llvm/pr-subscribers-clang
Changes
When a nested template is instantiated, the template pattern of the inner class
is not copied into the outer class ClassTemplateSpecializationDecl. The
specialization contains a ClassTemplateDecl with an empty record that points to
the
https://github.com/antangelo created
https://github.com/llvm/llvm-project/pull/68379
When a nested template is instantiated, the template pattern of the inner class
is not copied into the outer class ClassTemplateSpecializationDecl. The
specialization contains a ClassTemplateDecl with an
shiltian wrote:
> Does `thread_limit` directly imply the number of threads? I thought that it
> merely set an upper bound such that it cannot be increased beyond that via
> environment variables.
Based on the spec, yes. However, here since `ompx_bare` is an extension, we can
redefine
https://github.com/Endilll updated
https://github.com/llvm/llvm-project/pull/68377
>From 8503ff3e70e585a4a3597e3ee8111269f48105cc Mon Sep 17 00:00:00 2001
From: Vlad Serebrennikov
Date: Thu, 5 Oct 2023 13:18:14 +0300
Subject: [PATCH 1/2] [clang] Align Type and ExtQuals on 16-byte boundary
---
Endilll wrote:
Relevant code snippets:
https://github.com/llvm/llvm-project/blob/777a6e6f10b2b90496d248b7fa904fce834484be/clang/lib/AST/ASTContext.cpp#L5813C1-L5816C33
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 da0ce32cc3221686f30a6267f43d8c5dd469ef80
8503ff3e70e585a4a3597e3ee8111269f48105cc --
Endilll wrote:
CC @zygoloid as the author of ee0ce302c5eb4f26738f334f2fd8b165fa65dfca
https://github.com/llvm/llvm-project/pull/68377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Endilll wrote:
My plan is to phase-out `TypeAlignment` in subsequent patches, because we have
proper language support for `alignas` and `alignof` now. It's the only one of
its kind passed to `Allocate()` (I inspected all references to this function).
CC @zygolod as the author of
llvmbot wrote:
@llvm/pr-subscribers-clang
Changes
While working on LLDB visualizer for `QualType`, I stumbled upon `Type` and
`ExtQuals` defined with `alignas(8)`. Such alignment leaves only 3 lower bits
available for pointer tagging, whereas `QualType` requires 4 (3 qualifiers +
https://github.com/Endilll created
https://github.com/llvm/llvm-project/pull/68377
While working on LLDB visualizer for `QualType`, I stumbled upon `Type` and
`ExtQuals` defined with `alignas(8)`. Such alignment leaves only 3 lower bits
available for pointer tagging, whereas `QualType`
jhuber6 wrote:
Does `thread_limit` directly imply the number of threads? I thought that it
merely set an upper bound such that it cannot be increased beyond that via
environment variables.
https://github.com/llvm/llvm-project/pull/68373
___
https://github.com/aartbik updated
https://github.com/llvm/llvm-project/pull/68360
>From 6094912685a0cfa5c13e023e8ec97238a84fca2f Mon Sep 17 00:00:00 2001
From: Aart Bik
Date: Thu, 5 Oct 2023 13:22:28 -0700
Subject: [PATCH 1/4] [mlir][sparse] introduce MapRef, unify conversion/codegen
for
Author: Owen Pan
Date: 2023-10-05T19:24:08-07:00
New Revision: 84cbd9f4ffba17a74516d1afa442568dc54eabb1
URL:
https://github.com/llvm/llvm-project/commit/84cbd9f4ffba17a74516d1afa442568dc54eabb1
DIFF:
https://github.com/llvm/llvm-project/commit/84cbd9f4ffba17a74516d1afa442568dc54eabb1.diff
https://github.com/JOE1994 updated
https://github.com/llvm/llvm-project/pull/68277
>From 2c3fb032e7fdefa553ed4833bbeed8852eb6a410 Mon Sep 17 00:00:00 2001
From: Youngsuk Kim
Date: Mon, 2 Oct 2023 12:28:33 -0400
Subject: [PATCH 1/3] [clang] Replace uses of
CreatePointerBitCastOrAddrSpaceCast
https://github.com/JOE1994 updated
https://github.com/llvm/llvm-project/pull/68277
>From 2c3fb032e7fdefa553ed4833bbeed8852eb6a410 Mon Sep 17 00:00:00 2001
From: Youngsuk Kim
Date: Mon, 2 Oct 2023 12:28:33 -0400
Subject: [PATCH 1/2] [clang] Replace uses of
CreatePointerBitCastOrAddrSpaceCast
https://github.com/ThomasRaoux approved this pull request.
Looks like it required quite a lot of cases to be handled :(
Thanks for doing this, it solves some of the problems triton had with latest
LLVM. Changes look good to me.
https://github.com/llvm/llvm-project/pull/67866
https://github.com/ThomasRaoux edited
https://github.com/llvm/llvm-project/pull/67866
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,1248 @@
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
UTC_ARGS: --version 3
+; ## Support i16x2 instructions
+; RUN: llc < %s -mtriple=nvptx64-nvidia-cuda -mcpu=sm_90 -mattr=+ptx80 \
+; RUN: -O0 -disable-post-ra
https://github.com/ThomasRaoux approved this pull request.
Looks like it required quite a lot of cases to be handled :(
Thanks for doing this, it solves some of the problems triton had with latest
LLVM. Changes look good to me.
https://github.com/llvm/llvm-project/pull/67866
llvmbot wrote:
@llvm/pr-subscribers-clang
Changes
This patch makes `num_teams` and `thread_limit` mandatory for bare kernels,
similar to a reguar kernel language that when launching a kernel, the grid size
has to be set explicitly.
---
Patch is 34.10 KiB, truncated to 20.00 KiB below,
https://github.com/shiltian created
https://github.com/llvm/llvm-project/pull/68373
This patch makes `num_teams` and `thread_limit` mandatory for bare kernels,
similar to a reguar kernel language that when launching a kernel, the grid size
has to be set explicitly.
>From
ThomasRaoux wrote:
I ran the patch on our triton kernels and I don't see any functional problems
left.
https://github.com/llvm/llvm-project/pull/67866
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/ahatanak edited
https://github.com/llvm/llvm-project/pull/67454
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ahatanak edited
https://github.com/llvm/llvm-project/pull/67454
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/s-barannikov approved this pull request.
LGTM with the suggestion applied.
https://github.com/llvm/llvm-project/pull/68277
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -3088,9 +3088,6 @@ void ItaniumCXXABI::EmitThreadLocalInitFuncs(
CharUnits Align = CGM.getContext().getDeclAlign(VD);
Val = Builder.CreateAlignedLoad(Var->getValueType(), Val, Align);
}
-if (Val->getType() != Wrapper->getReturnType())
- Val =
https://github.com/kasuga-fj updated
https://github.com/llvm/llvm-project/pull/65246
>From 01ca65833b2248547f1fb4f7d739517e619ac4cd Mon Sep 17 00:00:00 2001
From: "kasuga.ryotaro"
Date: Wed, 30 Aug 2023 13:26:31 +0900
Subject: [PATCH] [Clang] Fix missing diagnostic for non-standard layout type
https://github.com/luamfb created
https://github.com/llvm/llvm-project/pull/68368
Fix issue https://github.com/llvm/llvm-project/issues/66990
>From 1f0249fdb0cf7b1799eb9e532bfbb4a3ffe983b0 Mon Sep 17 00:00:00 2001
From: luamfb
Date: Thu, 5 Oct 2023 21:50:10 -0300
Subject: [PATCH] [analyzer]
ZijunZhao updated this revision to Diff 557622.
ZijunZhao added a comment.
1. set `--triple=x86-64` as the first argument
2. run the script for the tests
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157331/new/
https://reviews.llvm.org/D157331
jrtc27 added inline comments.
Comment at: clang/test/C/C2x/n2683_2.c:5-6
+#include
+#include
+// CHECK-LABEL: define dso_local void @test_add_overflow_to64(
+// CHECK-SAME: ) #[[ATTR0:[0-9]+]] {
UTC won't insert such a blank line if it doesn't exist in the
joker-eph wrote:
Reverted, the bots have been broken all day.
https://github.com/llvm/llvm-project/pull/68136
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,51 @@
+#include "mlir/Analysis/Presburger/Fraction.h"
+#include "./Utils.h"
+#include
+#include
+
+using namespace mlir;
+using namespace presburger;
+
+TEST(FractionTest, getAsInteger) {
+ Fraction f(3, 1);
+ EXPECT_EQ(f.getAsInteger(), MPInt(3));
+}
+
https://github.com/Superty requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/68298
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,51 @@
+#include "mlir/Analysis/Presburger/Fraction.h"
+#include "./Utils.h"
+#include
+#include
+
+using namespace mlir;
+using namespace presburger;
+
+TEST(FractionTest, getAsInteger) {
+ Fraction f(3, 1);
+ EXPECT_EQ(f.getAsInteger(), MPInt(3));
+}
+
https://github.com/Superty edited
https://github.com/llvm/llvm-project/pull/68298
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,51 @@
+#include "mlir/Analysis/Presburger/Fraction.h"
+#include "./Utils.h"
+#include
+#include
+
+using namespace mlir;
+using namespace presburger;
+
+TEST(FractionTest, getAsInteger) {
+ Fraction f(3, 1);
+ EXPECT_EQ(f.getAsInteger(), MPInt(3));
+}
+
@@ -0,0 +1,51 @@
+#include "mlir/Analysis/Presburger/Fraction.h"
+#include "./Utils.h"
+#include
+#include
+
+using namespace mlir;
+using namespace presburger;
+
+TEST(FractionTest, getAsInteger) {
+ Fraction f(3, 1);
+ EXPECT_EQ(f.getAsInteger(), MPInt(3));
+}
+
https://github.com/Superty requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/68298
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3088,9 +3088,6 @@ void ItaniumCXXABI::EmitThreadLocalInitFuncs(
CharUnits Align = CGM.getContext().getDeclAlign(VD);
Val = Builder.CreateAlignedLoad(Var->getValueType(), Val, Align);
}
-if (Val->getType() != Wrapper->getReturnType())
- Val =
@@ -2138,6 +2138,8 @@ class IRBuilderBase {
return Insert(CastInst::CreatePointerCast(V, DestTy), Name);
}
+ // With opaque pointers enabled, this is same as CreateAddressSpaceCast.
JOE1994 wrote:
```suggestion
// With opaque pointers enabled, this
@@ -2138,6 +2138,8 @@ class IRBuilderBase {
return Insert(CastInst::CreatePointerCast(V, DestTy), Name);
}
+ // With opaque pointers enabled, this is same as CreateAddressSpaceCast.
JOE1994 wrote:
> If the types (= address spaces) are the same,
>
https://github.com/rjmccall edited
https://github.com/llvm/llvm-project/pull/68344
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -156,7 +156,8 @@ ImplicitConversionRank
clang::GetConversionRank(ImplicitConversionKind Kind) {
// it was omitted by the patch that added
// ICK_Zero_Queue_Conversion
ICR_C_Conversion,
-ICR_C_Conversion_Extension
+
@@ -2192,6 +2194,9 @@ static bool IsStandardConversion(Sema , Expr* From,
QualType ToType,
From->isIntegerConstantExpr(S.getASTContext())) {
SCS.Second = ICK_Compatible_Conversion;
FromType = ToType;
+ } else if (ToType->isFixedPointType() ||
@@ -2138,6 +2138,8 @@ class IRBuilderBase {
return Insert(CastInst::CreatePointerCast(V, DestTy), Name);
}
+ // With opaque pointers enabled, this is same as CreateAddressSpaceCast.
s-barannikov wrote:
I'll try to rephrase @arsenm comment.
If the
@@ -3088,9 +3088,6 @@ void ItaniumCXXABI::EmitThreadLocalInitFuncs(
CharUnits Align = CGM.getContext().getDeclAlign(VD);
Val = Builder.CreateAlignedLoad(Var->getValueType(), Val, Align);
}
-if (Val->getType() != Wrapper->getReturnType())
- Val =
huangjd wrote:
It looks like after using HashKeyMap, the purpose of SymbolMap and
GUIDToFuncNameMap can be merged. SymbolMap maps StringRef to Function*,
GUIDToFuncNameMap maps ProfileFuncRef's hash code to StringRef, and it's almost
exclusively cascaded to SymbolMap lookup, so we can
huangjd wrote:
It looks like after using HashKeyMap, the purpose of SymbolMap and
GUIDToFuncNameMap can be merged. SymbolMap maps StringRef to Function*,
GUIDToFuncNameMap maps ProfileFuncRef's hash code to StringRef, and it's almost
exclusively cascaded to SymbolMap lookup, so we can
https://github.com/modiking 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
@@ -19,20 +19,38 @@ namespace llvm {
// Visitor class that finds all indirect call.
struct PGOIndirectCallVisitor : public InstVisitor {
std::vector IndirectCalls;
+ std::vector VTableAddrs;
PGOIndirectCallVisitor() = default;
void visitCallBase(CallBase ) {
if
llvmbot wrote:
@llvm/pr-subscribers-flang-openmp
Changes
This patch adds support for `thread_limit` and upperbound on `num_teams` clause
for the teams construct in OpenMP.
Added testcases for the same.
---
Full diff: https://github.com/llvm/llvm-project/pull/68364.diff
3 Files
llvmbot wrote:
@llvm/pr-subscribers-flang-openmp
Changes
This patch adds support for `thread_limit` and upperbound on `num_teams` clause
for the teams construct in OpenMP.
Added testcases for the same.
---
Full diff: https://github.com/llvm/llvm-project/pull/68364.diff
3 Files
https://github.com/vabridgers edited
https://github.com/llvm/llvm-project/pull/68276
___
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-openmp
Changes
This patch adds support for `thread_limit` and upperbound on `num_teams` clause
for the teams construct in OpenMP.
Added testcases for the same.
---
Full diff: https://github.com/llvm/llvm-project/pull/68364.diff
3 Files
https://github.com/shraiysh created
https://github.com/llvm/llvm-project/pull/68364
This patch adds support for `thread_limit` and upperbound on `num_teams` clause
for the teams construct in OpenMP.
Added testcases for the same.
>From 2d3b34476df53f39d6cc6b7eee02b9d0d33e7a04 Mon Sep 17
https://github.com/shraiysh created
https://github.com/llvm/llvm-project/pull/68364
This patch adds support for `thread_limit` and upperbound on `num_teams` clause
for the teams construct in OpenMP.
Added testcases for the same.
>From 2d3b34476df53f39d6cc6b7eee02b9d0d33e7a04 Mon Sep 17
https://github.com/shraiysh created
https://github.com/llvm/llvm-project/pull/68364
This patch adds support for `thread_limit` and upperbound on `num_teams` clause
for the teams construct in OpenMP.
Added testcases for the same.
>From 2d3b34476df53f39d6cc6b7eee02b9d0d33e7a04 Mon Sep 17
https://github.com/vabridgers updated
https://github.com/llvm/llvm-project/pull/68276
>From 83dd0fa7956942b6cffefb12caf5f070196b4ca3 Mon Sep 17 00:00:00 2001
From: Vince Bridgers
Date: Thu, 5 Oct 2023 02:39:12 +0200
Subject: [PATCH] [Sema] Add check for bitfield assignments to integral types
https://github.com/brad0 updated https://github.com/llvm/llvm-project/pull/68349
>From c333fbefacc648600fbf84466a60a20cb2331fc3 Mon Sep 17 00:00:00 2001
From: Brad Smith
Date: Thu, 5 Oct 2023 16:09:04 -0400
Subject: [PATCH] [Driver] Group together usage of AddAllArgs
---
MasterAwesome wrote:
> > > > @erichkeane @AaronBallman, I've added the release notes and rebased the
> > > > change on `main`. Does this need to be backported to 17.x? 16.x doesn't
> > > > have this issue
> > >
> > >
> > > Yeah, I think we should probably backport this to 17.x. We have
> >
jansvoboda11 wrote:
> > I was going to suggest keeping Header::PathRelativeToRootModuleDirectory as
> > "float.h", and generating this into the buffer
>
> Wouldn't this work without the pragma? Isn't the resource directory always
> added to the header search paths, so the header should be
Author: Joseph Huber
Date: 2023-10-05T17:10:03-05:00
New Revision: 1d959f932752b39cd535cb22f04e348a4b01
URL:
https://github.com/llvm/llvm-project/commit/1d959f932752b39cd535cb22f04e348a4b01
DIFF:
https://github.com/llvm/llvm-project/commit/1d959f932752b39cd535cb22f04e348a4b01.diff
https://github.com/jhuber6 closed
https://github.com/llvm/llvm-project/pull/68264
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aartbik wrote:
This broke the bot?
https://lab.llvm.org/buildbot/#/builders/61/builds/50100
https://github.com/llvm/llvm-project/pull/68136
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/michaelrj-google updated
https://github.com/llvm/llvm-project/pull/68134
>From baf9d8e19d2b064c05527757c6173f875b59b286 Mon Sep 17 00:00:00 2001
From: Michael Jones
Date: Tue, 3 Oct 2023 10:39:02 -0700
Subject: [PATCH 1/4] [clang-tidy][libc] Fix namespace check with macro
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Changes
The previous code was pretty messy and treated value dependant expressions
which could not be evaluated the same as if they evaluted to `false`. Which was
obviously not correct.
We now check if we can evaluate the dependant
https://github.com/AMS21 created https://github.com/llvm/llvm-project/pull/68359
The previous code was pretty messy and treated value dependant expressions
which could not be evaluated the same as if they evaluted to `false`. Which was
obviously not correct.
We now check if we can evaluate
@@ -45,18 +46,21 @@ void CalleeNamespaceCheck::check(const
MatchFinder::MatchResult ) {
if (FuncDecl->getBuiltinID() != 0)
return;
- // If the outermost namespace of the function is __llvm_libc, we're good.
+ // If the outermost namespace of the function is a macro
ZijunZhao updated this revision to Diff 557621.
ZijunZhao added a comment.
Add --triple=x86_64 into the tests.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157331/new/
https://reviews.llvm.org/D157331
Files:
clang/docs/ReleaseNotes.rst
ilg-ul wrote:
@jansvoboda11, as Martin showed before, Apple clang did not fix this, so this
patch applies to it too.
https://github.com/llvm/llvm-project/pull/68091
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/PiJoules updated
https://github.com/llvm/llvm-project/pull/68344
>From 1eb1638a12c619febf1fe9830f34e51a56d4c20e Mon Sep 17 00:00:00 2001
From: Leonard Chan
Date: Thu, 5 Oct 2023 19:19:47 +
Subject: [PATCH] [clang] Ensure fixed point conversions work in C++
---
@@ -1,6 +1,8 @@
// NOTE: Assertions have been autogenerated by utils/update_cc_test_checks.py
-// RUN: %clang_cc1 -ffixed-point -triple x86_64-unknown-linux-gnu -S
-emit-llvm %s -o - | FileCheck %s --check-prefixes=CHECK,SIGNED
-// RUN: %clang_cc1 -ffixed-point -triple
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/68264
>From 5010ba6ec2f7370d2efc01d2fafb86d55c4dd883 Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Wed, 4 Oct 2023 16:50:20 -0500
Subject: [PATCH] [OpenMP] Prevent AMDGPU from overriding visibility on
DT_nohost
https://github.com/jdoerfert approved this pull request.
LG if others don't object
https://github.com/llvm/llvm-project/pull/68264
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jdoerfert wrote:
Seems reasonable to me.
https://github.com/llvm/llvm-project/pull/68264
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -45,18 +46,21 @@ void CalleeNamespaceCheck::check(const
MatchFinder::MatchResult ) {
if (FuncDecl->getBuiltinID() != 0)
return;
- // If the outermost namespace of the function is __llvm_libc, we're good.
+ // If the outermost namespace of the function is a macro
@@ -0,0 +1,302 @@
+//===--- cc1modbuildd_main.cpp - Clang CC1 Module Build Daemon
===//
+//
+// 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:
https://github.com/PiotrZSL approved this pull request.
>From functional point of view looks fine, could be more strict but should
>still work.
https://github.com/llvm/llvm-project/pull/68134
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/68134
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/68134
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/68134
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cachemeifyoucan commented:
I haven't read the PR in details but one thing we found difficult to do is how
to write a test for daemon spawning logic, because if anything went wrong, you
left an unbounded daemon process on CI node.
The strategy in this PR (try to kill it
jansvoboda11 wrote:
> @jansvoboda11, since the Apple clang already fixed this bug, could you
> suggest how to proceed? Could you backport the Apple patch to upstream?
I'm not familiar with this, but @yln might.
https://github.com/llvm/llvm-project/pull/68091
@@ -45,18 +46,21 @@ void CalleeNamespaceCheck::check(const
MatchFinder::MatchResult ) {
if (FuncDecl->getBuiltinID() != 0)
return;
- // If the outermost namespace of the function is __llvm_libc, we're good.
+ // If the outermost namespace of the function is a macro
@@ -45,18 +46,21 @@ void CalleeNamespaceCheck::check(const
MatchFinder::MatchResult ) {
if (FuncDecl->getBuiltinID() != 0)
return;
- // If the outermost namespace of the function is __llvm_libc, we're good.
+ // If the outermost namespace of the function is a macro
@@ -45,6 +45,10 @@ class GlobalCompilationDatabase {
return std::nullopt;
}
+ virtual std::vector getAllFilesInProjectOf(PathRef File) const {
mathstuf wrote:
`compile_commands.json` probably isn't the best place for it as it cannot be
filled out at
https://github.com/brad0 updated https://github.com/llvm/llvm-project/pull/68349
>From c4a869323ede651b71dc514199cee112279b0ce9 Mon Sep 17 00:00:00 2001
From: Brad Smith
Date: Thu, 5 Oct 2023 16:09:04 -0400
Subject: [PATCH] [Driver] Group together usage of AddAllArgs
---
@@ -45,18 +46,21 @@ void CalleeNamespaceCheck::check(const
MatchFinder::MatchResult ) {
if (FuncDecl->getBuiltinID() != 0)
return;
- // If the outermost namespace of the function is __llvm_libc, we're good.
+ // If the outermost namespace of the function is a macro
https://github.com/brad0 updated https://github.com/llvm/llvm-project/pull/68349
>From 74ad5f7b60f7e78e524311c5dbbce16650d9d890 Mon Sep 17 00:00:00 2001
From: Brad Smith
Date: Thu, 5 Oct 2023 16:09:04 -0400
Subject: [PATCH] [Driver] Group together usage of AddAllArgs
---
rmaz wrote:
> I thought you're trying to fix the situation where your resource dir is in
> the CWD, so that's the context I'm assuming. Is that correct?
Yes.
> I was considering your test case, where we'd previously store
> "resource-dir/include/float.h" into
https://github.com/brad0 updated https://github.com/llvm/llvm-project/pull/68349
>From 6542f6aae993dd186f736da7d3d2fcb54647f01f Mon Sep 17 00:00:00 2001
From: Brad Smith
Date: Thu, 5 Oct 2023 16:09:04 -0400
Subject: [PATCH] [Driver] Group together usage of AddAllArgs
---
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Changes
MaskRay suggested removal of the identifer in a review for another ToolChain so
doing the same for the rest.
---
Full diff: https://github.com/llvm/llvm-project/pull/68351.diff
8 Files Affected:
- (modified)
https://github.com/brad0 created https://github.com/llvm/llvm-project/pull/68351
MaskRay suggested removal of the identifer in a review for another ToolChain so
doing the same for the rest.
>From 977aaf66e606f454949edae1d18b14cb06307a51 Mon Sep 17 00:00:00 2001
From: Brad Smith
Date: Thu, 5
1 - 100 of 373 matches
Mail list logo