https://github.com/minglotus-6 edited
https://github.com/llvm/llvm-project/pull/70841
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay wrote:
I am unfamiliar with Darwin and @jansvoboda11 @ldionne can give a more
authoritative answer, but I can share some feelings.
* Gnu.cpp uses `Generic_GCC::addLibCxxIncludePaths` so other versions (e.g.
`usr/include/c++/v2`) are supported.
* Only one of `Dir`/`InstalledDir` should
@@ -279,12 +279,13 @@ class DefaultIntArgument :
IntArgument {
// This argument is more complex, it includes the enumerator type name,
// a list of strings to accept, and a list of enumerators to map them to.
-class EnumArgument values,
+class EnumArgument values,
---
@@ -1010,14 +1022,17 @@ namespace {
// trivial because some enumeration values have multiple named
// enumerators, such as type_visibility(internal) and
// type_visibility(hidden) both mapping to TypeVisibilityAttr::Hidden.
- OS << "const char *" << getAt
https://github.com/leo-ard created
https://github.com/llvm/llvm-project/pull/70845
This PR fixes https://github.com/llvm/llvm-project/issues/55013 : the max
intrinsics is not generated for this simple loop case :
https://godbolt.org/z/hxz1xhMPh. This is caused by a ICMP not being folded into
@@ -1222,6 +1222,15 @@ void ExprEngine::ProcessInitializer(const CFGInitializer
CFGInit,
PostInitializer PP(BMI, FieldLoc.getAsRegion(), stackFrame);
evalBind(Tmp, Init, Pred, FieldLoc, InitVal, /*isInit=*/true, &PP);
}
+ } else if (BMI->isBaseInitializer() &&
llvmbot wrote:
@llvm/pr-subscribers-backend-x86
Author: Léonard Oest O'Leary (leo-ard)
Changes
This PR fixes https://github.com/llvm/llvm-project/issues/55013 : the max
intrinsics is not generated for this simple loop case :
https://godbolt.org/z/hxz1xhMPh. This is caused by a ICMP not b
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 a9b3f200159595ed8f13cc891ae34291020ba111
99abebc88c09346bd4a70bbf39df13b249116c09 --
https://github.com/leo-ard closed
https://github.com/llvm/llvm-project/pull/67594
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
leo-ard wrote:
I'm closing this PR in favour of #70845
https://github.com/llvm/llvm-project/pull/67594
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jhuber6 wrote:
Small change that affects a lot of tests. LG once these are fixed:
```
MLIR :: Target/LLVMIR/omptarget-region-device-llvm.mlir
MLIR :: Target/LLVMIR/omptarget-byref-bycopy-generation-device.mlir
MLIR :: Target/LLVMIR/omptarget-declare-target-llvm-device.mlir
```
https://github.com
@@ -279,12 +279,13 @@ class DefaultIntArgument :
IntArgument {
// This argument is more complex, it includes the enumerator type name,
// a list of strings to accept, and a list of enumerators to map them to.
-class EnumArgument values,
+class EnumArgument values,
---
https://github.com/steakhal approved this pull request.
Looks great!
FYI when submitting patches to GH try not to force-push to help the UI for
following lines having comments. Otherwise, they will be marked as "outdated"
and become hard to dig up and relate to new line locations. This was eas
@@ -279,12 +279,13 @@ class DefaultIntArgument :
IntArgument {
// This argument is more complex, it includes the enumerator type name,
// a list of strings to accept, and a list of enumerators to map them to.
-class EnumArgument values,
+class EnumArgument values,
---
@@ -898,23 +898,29 @@ namespace {
}
class EnumArgument : public Argument {
-std::string type;
+std::string fullType;
+StringRef shortType;
std::vector values, enums, uniques;
+bool isExternal;
public:
EnumArgument(const Record &Arg, StringRef
@@ -0,0 +1,19 @@
+// RUN: %clang_cc1 -triple x86_64 -emit-llvm %s -o - | FileCheck %s
+
+// CHECK: define dso_local noundef i32 @_Z4testi(i32 noundef %i) #0 {
+// CHECK-NEXT: entry:
+// CHECK-NEXT: %i.addr = alloca i32, align 4
+// CHECK-NEXT: store i32 %i, ptr %i.addr, a
https://github.com/yxsamliu updated
https://github.com/llvm/llvm-project/pull/70369
>From 5097f2075bc41a490a307c55b22dc2764c5b43f2 Mon Sep 17 00:00:00 2001
From: "Yaxun (Sam) Liu"
Date: Fri, 27 Oct 2023 23:34:51 -0400
Subject: [PATCH] [CUDA][HIP] Make template implicitly host device
Added opti
https://github.com/yxsamliu edited
https://github.com/llvm/llvm-project/pull/70369
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/yxsamliu updated
https://github.com/llvm/llvm-project/pull/70369
>From f68b605114ab6b7183b4516df3f1227ef5d6f9d8 Mon Sep 17 00:00:00 2001
From: "Yaxun (Sam) Liu"
Date: Fri, 27 Oct 2023 23:34:51 -0400
Subject: [PATCH] [CUDA][HIP] Make template implicitly host device
Added opti
https://github.com/yxsamliu edited
https://github.com/llvm/llvm-project/pull/70369
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/erichkeane commented:
Its a bit of a shame that the `VariadicEnumArgument` isn't being included
here/updated here, I typically appreciate the variadic versions being kept in
sync. But since we don't have a use for it at the moment, I think a 'fixme'
(or a future patch!) wou
https://github.com/steakhal updated
https://github.com/llvm/llvm-project/pull/70837
>From a7f64815f4986fad597b9cb2d1acce2de9ac20bf Mon Sep 17 00:00:00 2001
From: Balazs Benics
Date: Mon, 23 Oct 2023 18:10:29 +0200
Subject: [PATCH 1/2] [analyzer] Fix assertion failure in
CXXInstanceCall::getCXX
@@ -1010,14 +1022,17 @@ namespace {
// trivial because some enumeration values have multiple named
// enumerators, such as type_visibility(internal) and
// type_visibility(hidden) both mapping to TypeVisibilityAttr::Hidden.
- OS << "const char *" << getAt
https://github.com/erichkeane edited
https://github.com/llvm/llvm-project/pull/70835
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -930,77 +936,83 @@ namespace {
OS << getLowerName() << "(" << getUpperName() << ")";
}
void writeCtorDefaultInitializers(raw_ostream &OS) const override {
- OS << getLowerName() << "(" << type << "(0))";
+ OS << getLowerName() << "(" << fullType << "(
@@ -1015,7 +1018,8 @@ def ExternalSourceSymbol : InheritableAttr {
def Blocks : InheritableAttr {
let Spellings = [Clang<"blocks">];
- let Args = [EnumArgument<"Type", "BlockType", ["byref"], ["ByRef"]>];
+ let Args = [EnumArgument<"Type", "BlockType",
eri
steakhal wrote:
> Hmm, I wonder if we should leave a FIXME comment, but it looks good to me.
I was thinking where to put the FIXME, and as I explored that should be within
the CastVisitor. After that, I argued, that then I should still have the
(ineffective) `SVB.evalCast()` to actually exerci
@@ -265,6 +269,63 @@ bool AIXTargetCodeGenInfo::initDwarfEHRegSizeTable(
return PPC_initDwarfEHRegSizeTable(CGF, Address, Is64Bit, /*IsAIX*/ true);
}
+void AIXTargetCodeGenInfo::setTargetAttributes(
+const Decl *D, llvm::GlobalValue *GV, CodeGen::CodeGenModule &M) const
https://github.com/efriedma-quic edited
https://github.com/llvm/llvm-project/pull/67999
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/efriedma-quic commented:
What are the ABI implications here? Do all translation units have to agree on
whether a variable is "tocdata"? The manual change hints there's some sort of
linker fixup, but it's not clear to me how that works.
Specifying the names of global variab
@@ -622,6 +622,28 @@ static bool checkAliasedGlobal(
return true;
}
+// Emit a warning if toc-data attribute is requested for global variables that
+// have aliases and remove the toc-data attribute.
+static void checkAliasForTocData(llvm::GlobalVariable *GVar,
+
ilg-ul wrote:
> Only one of `Dir`/`InstalledDir` should be used
The current behaviour is:
- if `InstalledDir/../include/c++/v1` exists, it is used
- otherwise the `MacOSX.sdk/usr/include/c++/v1` is used.
In most cases, when invoked via a link, there is no `include` in that folder
(see the exa
https://github.com/david-xl approved this pull request.
https://github.com/llvm/llvm-project/pull/69513
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/david-xl edited
https://github.com/llvm/llvm-project/pull/69513
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -314,7 +314,22 @@ static void loadInput(const WeightedFile &Input,
SymbolRemapper *Remapper,
}
auto FS = vfs::getRealFileSystem();
- auto ReaderOrErr = InstrProfReader::create(Input.Filename, *FS, Correlator);
+ // TODO: This only saves the first non-fatal error from
https://github.com/steakhal edited
https://github.com/llvm/llvm-project/pull/70837
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/steakhal edited
https://github.com/llvm/llvm-project/pull/70837
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1010,14 +1022,17 @@ namespace {
// trivial because some enumeration values have multiple named
// enumerators, such as type_visibility(internal) and
// type_visibility(hidden) both mapping to TypeVisibilityAttr::Hidden.
- OS << "const char *" << getAt
Xazax-hun wrote:
> WDYT?
I like this! I hope we do not add too much redundant work, but at least we make
it clear what is the plan to fix this in the future.
https://github.com/llvm/llvm-project/pull/70837
___
cfe-commits mailing list
cfe-commits@lis
https://github.com/dwblaikie approved this pull request.
Looks great - thanks!
https://github.com/llvm/llvm-project/pull/70674
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -930,77 +936,83 @@ namespace {
OS << getLowerName() << "(" << getUpperName() << ")";
}
void writeCtorDefaultInitializers(raw_ostream &OS) const override {
- OS << getLowerName() << "(" << type << "(0))";
+ OS << getLowerName() << "(" << fullType << "(
Author: Vlad Serebrennikov
Date: 2023-10-31T21:50:18+03:00
New Revision: ae7b20b583fab1325d8b51fe5f2eaf612de8b95e
URL:
https://github.com/llvm/llvm-project/commit/ae7b20b583fab1325d8b51fe5f2eaf612de8b95e
DIFF:
https://github.com/llvm/llvm-project/commit/ae7b20b583fab1325d8b51fe5f2eaf612de8b95e.
@@ -67,15 +67,15 @@ int C::a = 4;
// CHECK-NOT:size:
// CHECK-NOT:align:
// CHECK-NOT:offset:
-// CHECK-SAME: flags: DIFlagStaticMember,
-// CHECK-SAME: extraData: i1 true)
+// CHECK-SAME: flags: DIFlagStaticMemb
efriedma-quic wrote:
The new option in CodeGenOptions.h seems fine.
If your goal is to statically inject passes into the clang executable, I don't
understand why the existing LLVM plugin infrastructure isn't sufficient for
your needs. You mention "modifying Extension.def", but that file is ge
egorzhdan wrote:
The clang-format failure is not related to the changes in this PR.
https://github.com/llvm/llvm-project/pull/70827
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -5883,6 +5907,18 @@ void CGDebugInfo::finalize() {
DBuilder.replaceTemporary(std::move(FwdDecl), cast(Repl));
}
+ for (auto const *VD : StaticDataMemberDefinitionsToEmit) {
+assert(VD->isStaticDataMember());
+
+if (auto It = DeclCache.find(VD); It != DeclCach
zygoloid wrote:
Thanks for looking into this!
I wonder if it would make sense to move the checking for this warning here, to
the `else` case in line 725:
https://github.com/llvm/llvm-project/blob/ac30780250875802d13450d17e6959f9e2ad3a70/clang/lib/Sema/SemaInit.cpp#L717-L734
That should remove
@@ -279,12 +279,13 @@ class DefaultIntArgument :
IntArgument {
// This argument is more complex, it includes the enumerator type name,
// a list of strings to accept, and a list of enumerators to map them to.
-class EnumArgument values,
+class EnumArgument values,
---
steakhal wrote:
> > WDYT?
>
> I like this! I hope we do not add too much redundant work, but at least we
> make it clear what is the plan to fix this in the future.
Please approve the PR again, so that I could merge this after I give some time
for others to look at this.
https://github.com/l
https://github.com/martinboehme created
https://github.com/llvm/llvm-project/pull/70848
This can make the flow condition significantly easier to interpret; see below
for an example.
I had hoped that adding the simplification as a preprocessing step before the
SAT solver (in `DataflowAnalysisCon
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (martinboehme)
Changes
This can make the flow condition significantly easier to interpret; see below
for an example.
I had hoped that adding the simplification as a preprocessing step before the
SAT solver (in `DataflowAnalysisContext
https://github.com/doshimili updated
https://github.com/llvm/llvm-project/pull/70493
>From 6c313955185c0d59564f6535b6f1580dca168bea Mon Sep 17 00:00:00 2001
From: Your Name
Date: Tue, 24 Oct 2023 21:15:23 +
Subject: [PATCH 1/9] Add software prefetching to memset
---
libc/src/string/memory
@@ -898,23 +898,29 @@ namespace {
}
class EnumArgument : public Argument {
-std::string type;
+std::string fullType;
+StringRef shortType;
std::vector values, enums, uniques;
+bool isExternal;
public:
EnumArgument(const Record &Arg, StringRef
craig.topper added a comment.
Need to update the table here
https://clang.llvm.org/docs/LanguageExtensions.html#vector-operations to
include RVV
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158259/new/
https://reviews.llvm.org/D158259
_
@@ -898,23 +898,29 @@ namespace {
}
class EnumArgument : public Argument {
-std::string type;
+std::string fullType;
+StringRef shortType;
std::vector values, enums, uniques;
+bool isExternal;
public:
EnumArgument(const Record &Arg, StringRef
https://github.com/efriedma-quic approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/66716
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
efriedma-quic wrote:
(Not sure if you have commit access; let me know if you want me to merge for
you.)
https://github.com/llvm/llvm-project/pull/66716
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/list
Timm =?utf-8?q?Bäder?= ,
Timm =?utf-8?q?Bäder?= ,
Timm =?utf-8?q?Bäder?= ,
Timm =?utf-8?q?Bäder?= ,
Timm =?utf-8?q?Bäder?= ,
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,816 @@
+// RUN: %clang_cc1 -verify -std=c++2a -fsyntax-only
-fexperimental-new-constant-inter
https://github.com/doshimili updated
https://github.com/llvm/llvm-project/pull/70493
>From 6c313955185c0d59564f6535b6f1580dca168bea Mon Sep 17 00:00:00 2001
From: Your Name
Date: Tue, 24 Oct 2023 21:15:23 +
Subject: [PATCH 01/10] Add software prefetching to memset
---
libc/src/string/memo
https://github.com/rampitec commented:
Also needed negative tests that gfx11-insts feature is required (using gfx1030
target for example) and for the immediate arguments. See for example
builtins-amdgcn-gfx11-err.cl and builtins-amdgcn-fp-atomics-gfx11-err.cl.
https://github.com/llvm/llvm-proj
gustedt wrote:
Hi,
yes the really bad choice here is by gcc to have the same name for basically
two different attributes.
For the value, they also missed the opportunity to do something sensible when
moving to C++ attributes, what a pitty.
For a concrete guideline in that jungle, when (= whi
erichkeane wrote:
> Hi, yes the really bad choice here is by gcc to have the same name for
> basically two different attributes. For the value, they also missed the
> opportunity to do something sensible when moving to C++ attributes, what a
> pitty. For a concrete guideline in that jungle, wh
https://github.com/egorzhdan updated
https://github.com/llvm/llvm-project/pull/70827
>From 552995265316fd04bd73148fce010aa3cd0368f5 Mon Sep 17 00:00:00 2001
From: Egor Zhdan
Date: Tue, 31 Oct 2023 16:39:45 +
Subject: [PATCH] [APINotes] Upstream APINotesOptions
This upstreams more of the Cl
https://github.com/HazardyKnusperkeks commented:
The code is okay, but I do not share the intention.
https://github.com/llvm/llvm-project/pull/70768
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo
https://github.com/HazardyKnusperkeks edited
https://github.com/llvm/llvm-project/pull/70768
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3117,9 +3117,16 @@ void UnwrappedLineParser::parseForOrWhileLoop(bool
HasParens) {
FormatTok->setFinalizedType(TT_ConditionLParen);
parseParens();
}
- // Event control.
- if (Style.isVerilog())
+
+ if (Style.isVerilog()) {
+// Event control.
parseVe
@@ -898,23 +898,29 @@ namespace {
}
class EnumArgument : public Argument {
-std::string type;
+std::string fullType;
+StringRef shortType;
std::vector values, enums, uniques;
+bool isExternal;
public:
EnumArgument(const Record &Arg, StringRef
https://github.com/doshimili closed
https://github.com/llvm/llvm-project/pull/70493
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
efriedma-quic wrote:
Do you need to check if the lambda is templated? For example, the following
doesn't compile:
```
auto GetNonCapturingLambda() { return [](){ return 42; }; }
auto f() { return GetNonCapturingLambda() == GetNonCapturingLambda(); }
```
https://github.com/llvm/llvm-project/pu
https://github.com/Xazax-hun approved this pull request.
https://github.com/llvm/llvm-project/pull/70837
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3553,6 +3553,48 @@ static unsigned getPackIndexForParam(Sema &S,
llvm_unreachable("parameter index would not be produced from template");
}
+// if `Specialization` is a `CXXConstructorDecl` or `CXXConversionDecl`,
+// we'll try to instantiate and update its explicit spec
https://github.com/shafik edited https://github.com/llvm/llvm-project/pull/70548
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/shafik approved this pull request.
I just want to make sure we have enough testing in place for this change but
overall I think I am happy.
https://github.com/llvm/llvm-project/pull/70548
___
cfe-commits mailing list
cfe-commits@lis
@@ -265,6 +269,63 @@ bool AIXTargetCodeGenInfo::initDwarfEHRegSizeTable(
return PPC_initDwarfEHRegSizeTable(CGF, Address, Is64Bit, /*IsAIX*/ true);
}
+void AIXTargetCodeGenInfo::setTargetAttributes(
+const Decl *D, llvm::GlobalValue *GV, CodeGen::CodeGenModule &M) const
@@ -265,6 +269,63 @@ bool AIXTargetCodeGenInfo::initDwarfEHRegSizeTable(
return PPC_initDwarfEHRegSizeTable(CGF, Address, Is64Bit, /*IsAIX*/ true);
}
+void AIXTargetCodeGenInfo::setTargetAttributes(
+const Decl *D, llvm::GlobalValue *GV, CodeGen::CodeGenModule &M) const
@@ -622,6 +622,28 @@ static bool checkAliasedGlobal(
return true;
}
+// Emit a warning if toc-data attribute is requested for global variables that
+// have aliases and remove the toc-data attribute.
+static void checkAliasForTocData(llvm::GlobalVariable *GVar,
+
@@ -859,53 +859,60 @@ CodeGenFunction::emitBuiltinObjectSize(const Expr *E,
unsigned Type,
}
if (IsDynamic) {
-LangOptions::StrictFlexArraysLevelKind StrictFlexArraysLevel =
-getLangOpts().getStrictFlexArraysLevel();
-const Expr *Base = E->IgnoreParenImpCa
https://github.com/bwendling updated
https://github.com/llvm/llvm-project/pull/70606
>From 19dd7db8ab5f98a618c717944c96b34e604fbc30 Mon Sep 17 00:00:00 2001
From: Bill Wendling
Date: Sun, 29 Oct 2023 14:58:04 -0700
Subject: [PATCH 1/7] [CodeGen] Revamp counted_by calculations
Break down the co
@@ -265,6 +269,63 @@ bool AIXTargetCodeGenInfo::initDwarfEHRegSizeTable(
return PPC_initDwarfEHRegSizeTable(CGF, Address, Is64Bit, /*IsAIX*/ true);
}
+void AIXTargetCodeGenInfo::setTargetAttributes(
+const Decl *D, llvm::GlobalValue *GV, CodeGen::CodeGenModule &M) const
https://github.com/diggerlin edited
https://github.com/llvm/llvm-project/pull/67999
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -859,53 +859,60 @@ CodeGenFunction::emitBuiltinObjectSize(const Expr *E,
unsigned Type,
}
if (IsDynamic) {
-LangOptions::StrictFlexArraysLevelKind StrictFlexArraysLevel =
-getLangOpts().getStrictFlexArraysLevel();
-const Expr *Base = E->IgnoreParenImpCa
https://github.com/rapidsna edited
https://github.com/llvm/llvm-project/pull/70606
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rapidsna edited
https://github.com/llvm/llvm-project/pull/70606
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Jake Egan
Date: 2023-10-31T16:23:05-04:00
New Revision: 4fc70197919cded66502d1ad853f9e7f344b9a77
URL:
https://github.com/llvm/llvm-project/commit/4fc70197919cded66502d1ad853f9e7f344b9a77
DIFF:
https://github.com/llvm/llvm-project/commit/4fc70197919cded66502d1ad853f9e7f344b9a77.diff
LOG
https://github.com/minglotus-6 edited
https://github.com/llvm/llvm-project/pull/70841
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/minglotus-6 edited
https://github.com/llvm/llvm-project/pull/70841
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -859,53 +859,60 @@ CodeGenFunction::emitBuiltinObjectSize(const Expr *E,
unsigned Type,
}
if (IsDynamic) {
-LangOptions::StrictFlexArraysLevelKind StrictFlexArraysLevel =
-getLangOpts().getStrictFlexArraysLevel();
-const Expr *Base = E->IgnoreParenImpCa
https://github.com/bwendling edited
https://github.com/llvm/llvm-project/pull/70606
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bwendling edited
https://github.com/llvm/llvm-project/pull/70606
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaggieYingYi updated
https://github.com/llvm/llvm-project/pull/65268
>From 5b750383cace7db24144d16ce8a9ca0864c37613 Mon Sep 17 00:00:00 2001
From: Ying Yi
Date: Fri, 1 Sep 2023 15:30:44 +0100
Subject: [PATCH 1/2] [Clang] Add two time-trace scope variables.
A time trace scope
https://github.com/ZequanWu created
https://github.com/llvm/llvm-project/pull/70856
Refactor some code from https://github.com/llvm/llvm-project/pull/69493.
#70712 was reverted due to linking failures. So, I removed
`-profile-correlate=` flag and kept `-debug-info-correlate` in this change.
llvmbot wrote:
@llvm/pr-subscribers-pgo
Author: Zequan Wu (ZequanWu)
Changes
Refactor some code from https://github.com/llvm/llvm-project/pull/69493.
#70712 was reverted due to linking failures. So, I removed
`-profile-correlate=` flag and kept `-debug-info-correlate` in this change.
-
https://github.com/ZequanWu updated
https://github.com/llvm/llvm-project/pull/69513
>From 9a1af6e1d47ab622979796f2319edec8a9c77928 Mon Sep 17 00:00:00 2001
From: Zequan Wu
Date: Wed, 18 Oct 2023 16:28:30 -0400
Subject: [PATCH 1/7] [llvm-profdata] Emit error when counter value is greater
than 2
https://github.com/ZequanWu closed
https://github.com/llvm/llvm-project/pull/69513
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ZequanWu wrote:
Sorry for so many noise regarding this change.
https://github.com/llvm/llvm-project/pull/70856
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaggieYingYi wrote:
Hi @AaronBallman,
Thanks so much for the explanation using an example. I now understand. I have
updated the patch in the commit
https://github.com/llvm/llvm-project/pull/65268/commits/f66e7d2b720692e9bc4bbc915de286f6de87e291.
Kind regards,
Maggie
https://github.com/llvm/l
https://github.com/clayborg commented:
One question I have here: where will the DW_TAG_variable get emitted for these
`constexpr`? For actual static member variables we emit a single
DW_TAG_variable in the file that declares the global variable, but for
`constexpr` we won't be able to do this
@@ -3618,6 +3618,30 @@ void CompilerInvocationBase::GenerateLangArgs(const
LangOptions &Opts,
GenerateArg(Consumer, OPT_frandomize_layout_seed_EQ, Opts.RandstructSeed);
}
+static void CheckBoundsSafetyLang(InputKind IK, DiagnosticsEngine &Diags) {
+ // Currently, bounds
bogner wrote:
> Its a bit of a shame that the `VariadicEnumArgument` isn't being included
> here/updated here, I typically appreciate the variadic versions being kept in
> sync. But since we don't have a use for it at the moment, I think a 'fixme'
> (or a future patch!) would be acceptable.
I
https://github.com/benlangmuir approved this pull request.
https://github.com/llvm/llvm-project/pull/69975
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
301 - 400 of 548 matches
Mail list logo
