___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
don't
thisk these scalars belong in `FloatType` and that the long-term solution is to
allow arbitrary newtypes / tags on bits / ... up there)
https://github.com/llvm/llvm-project/pull/94735
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https
_assign_const` that’s
emitted below here, should *not* be moved into the helper imo.)
https://github.com/llvm/llvm-project/pull/94159
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sirraide edited
https://github.com/llvm/llvm-project/pull/94159
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sirraide commented:
This is already looking better.
(Also, for the record, this is just a me thing and not official policy or
anything, but I’d find it easier if you didn’t force-push because then I’d have
an easier time telling what’s changed in recent commits; everything
llvm/llvm-project/pull/94494
_______
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
because you are subscribed to the Google Groups
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to jenkinsci-commits+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/jenkins
nBreakpoint` then this might be overkill and checking the optional
is sufficient and marginally more efficient.
https://github.com/llvm/llvm-project/pull/94625
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
device__|__global__|__host__|__host__
__device__}1 function. "
+ "It will cause warning with nvcc">,
yxsamliu wrote:
will modify the diagnostic message
https://github.com/llvm/llvm-project/pull/77359
___
cfe-commits
[TuneDLenFactor2] to the tuning list.
https://github.com/llvm/llvm-project/pull/94564
_______
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
/llvm/llvm-project/pull/94564
_______
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/preames edited
https://github.com/llvm/llvm-project/pull/94564
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/preames requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/94564
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ThomasRaoux commented:
Looks good to me once the other comments are addressed. Thanks for the patch.
https://github.com/llvm/llvm-project/pull/94735
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi
dll attribute logic has been the same between
GetOrCreateLLVMGlobal and GetOrCreateLLVMFunction so far, we should keep it the
same until we find an explicit reason to make them different.
https://github.com/llvm/llvm-project/pull/94664
_______
cfe-com
think this is
fine provided that `*@selector(dealloc) = s` is valid in Objective-C (as I
said, I don’t know Objective-C...)
https://github.com/llvm/llvm-project/pull/94159
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi
However, if you feel it would be better that both use the
same logic, then I will go ahead with adding this.
https://github.com/llvm/llvm-project/pull/94664
___________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
D->getNameAsString() <<
"\"\n";
-}
+}
JDevlieghere wrote:
Unrelated whitespace change?
https://github.com/llvm/llvm-project/pull/94208
_______
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/JDevlieghere approved this pull request.
https://github.com/llvm/llvm-project/pull/94208
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/JDevlieghere edited
https://github.com/llvm/llvm-project/pull/94208
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/trcrsired updated
https://github.com/llvm/llvm-project/pull/79667
>From 4f1ce895254dd9505150c1f5f5cb77454b9aca68 Mon Sep 17 00:00:00 2001
From: cqwrteur <100043421+trcrsi...@users.noreply.github.com>
Date: Sat, 1 Jun 2024 02:55:50 -0400
Subject: [PATCH]
https://github.com/jcsxky ready_for_review
https://github.com/llvm/llvm-project/pull/94725
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
.
+ // There is no representation for Inf or NaN.
+ NoNanInf,
arsenm wrote:
Invert and call SupportsNonFinite?
https://github.com/llvm/llvm-project/pull/94735
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https
= fopen("/home/test", "rb+");
+ if (fp) {
+// 3*5: 15 bytes read; which is not exactly 4 integers, thus we invalidate
the whole buffer.
balazske wrote:
This changes the check for uninitialized `c[4+13]`? But probably only indexes
4, 8, 12, 16 in `c` (start of the integer values) are marked as initialized,
and c[4+9] would be uninitialized too.
https://github.com/llvm/llvm-project/pull/93408
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ory = fcInfinity;
+ return (opStatus)(opOverflow | opInexact);
kuhar wrote:
nit: prefer `static_cast`
https://github.com/llvm/llvm-project/pull/94735
___________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
`Finite` or would that
be too overloaded?
https://github.com/llvm/llvm-project/pull/94735
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vm-project/pull/94735
_______
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sdkrystian approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/94756
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
_______
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
/pull/92957
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
not suggesting you go back and do it for this
patch, but I think it would have been helpful to pre-clang-format the file, and
offload more of the minor changes into a previous patch.
https://github.com/llvm/llvm-project/pull/92957
___
cfe-commits mailing list
cfe
,
+ /*S*/ nullptr);
mizvekov wrote:
```suggestion
/*S=*/nullptr);
```
https://github.com/llvm/llvm-project/pull/92957
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https
https://github.com/mizvekov edited
https://github.com/llvm/llvm-project/pull/92957
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
causing the call to not be
elided, but I can't see any calls other than `new` and `delete` in the code you
posted either.
https://github.com/llvm/llvm-project/pull/94670
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https
jcsxky wrote:
crash on trunk assertion.
https://github.com/llvm/llvm-project/pull/94725
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jcsxky edited https://github.com/llvm/llvm-project/pull/94725
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
int arg = 0;
+static const int arg = 0; // expected-note{{referenced member 'arg' is
declared here}}
};
- A::C::x a;
+ A::C::x a; // expected-note{{in instantiation of member class
'PR12884_original::A::C' requested here}}
}
namespace PR12884_half_fixed {
template struct A {
ere}}
}
namespace PR12884_half_fixed {
template struct A {
_______
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
FILE *fp = fopen("/home/test", "rb+");
+ if (fp) {
+// 3*5: 15 bytes read; which is not exactly 4 integers, thus we invalidate
the whole buffer.
balazske wrote:
I meant to add a `buffer[5] = 6;` at the begin and use
`clang_analyzer_dump(buffer[5]);` after the read. If a value is initialized
before the read it should not change to uninitialized.
https://github.com/llvm/llvm-project/pull/93408
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vm-project/pull/94735
_______
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/erichkeane approved this pull request.
https://github.com/llvm/llvm-project/pull/94756
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mizvekov wrote:
@kadircet @yozhu FYI this fixes the problem you reported.
https://github.com/llvm/llvm-project/pull/94756
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
com/llvm/llvm-project/pull/94756
_______
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
+ template class foo {};
+
+ template class MapType, typename Value>
+ Value bar(MapType map);
+
+ template class MapType, typename Value>
+ Value bar(MapType> map);
+
+ void aux() {
+map> input;
+bar(input);
+ }
+} // namespace regression1
_____
::Float6E3M2FN(),
false);
+ EXPECT_TRUE(SmallestDenorm.isDenormal());
+ EXPECT_EQ(0x0.1p0, SmallestDenorm.convertToFloat());
+}
+
+TEST(APFloatTest, Float6E2M3FNToFloat) {
+ APFloat PosZero = APFloat::getZero(APFloat::Float6E2M3FN());
+ APFloat PosZeroToFloat(PosZero.convertToFloat());
+ EXPE
ero/Largest/Smallest/SmallestNormalized} and
negative tests for get{Nan/Inf} cases.
Please let me know if I am still missing something.
https://github.com/llvm/llvm-project/pull/94735
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm
t6E2M3FN);
+}
durga4github wrote:
Sure, moved it as a static-helper inside APFloat and using that here
https://github.com/llvm/llvm-project/pull/94735
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin
::Float6E3M2FN(),
false);
+ EXPECT_TRUE(SmallestDenorm.isDenormal());
+ EXPECT_EQ(0x0.1p0, SmallestDenorm.convertToFloat());
+}
+
+TEST(APFloatTest, Float6E2M3FNToFloat) {
+ APFloat PosZero = APFloat::getZero(APFloat::Float6E2M3FN());
+ APFloat PosZeroToFloat(PosZero.convertToFloat());
+ EXPE
https://github.com/jhuber6 closed
https://github.com/llvm/llvm-project/pull/94751
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rogram(Program);
+ auto StdoutOrErr = executeToolChainProgram(Program, /*SecondsToWait=*/10);
if (!StdoutOrErr)
return StdoutOrErr.takeError();
_______
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ery simple and I will be posting a patch shortly.
https://github.com/llvm/llvm-project/pull/93433
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/arsenm approved this pull request.
https://github.com/llvm/llvm-project/pull/94751
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ChuanqiXu9 closed
https://github.com/llvm/llvm-project/pull/94603
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kadircet approved this pull request.
https://github.com/llvm/llvm-project/pull/92743
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
andreas pushed a change to branch wip-wget
in repository guix.
was e781fc526f gnu: wget: Update to 1.21.4.
This change permanently discards the following revisions:
discard e781fc526f gnu: wget: Update to 1.21.4.
erichkeane wrote:
I see the 2nds commit doesn't add any tests! Please make it do so, else LGTM
(plus might want to do a 'merge' commit to reset the CI to a more stable state).
https://github.com/llvm/llvm-project/pull/93913
___
cfe-commits mailing
erichkeane wrote:
I see the 2nds commit doesn't add any tests! Please make it do so, else LGTM
(plus might want to do a 'merge' commit to reset the CI to a more stable state).
https://github.com/llvm/llvm-project/pull/93913
___
lldb-commits mailing
/8c826be35c73...948f23417010
To unsubscribe from these emails, change your notification settings at
https://github.com/tianocore/edk2/settings/notifications
___
edk2-commits mailing list
edk2-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists
ait=*/10);
if (!StdoutOrErr)
return StdoutOrErr.takeError();
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/spaits edited https://github.com/llvm/llvm-project/pull/94752
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
enm wrote:
Name this SecondsToWait to match ExecuteAndWait?
https://github.com/llvm/llvm-project/pull/94751
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jhuber6 wrote:
No active test because I have no clue how you would, but I intentionally made
it time out and it returns a 'Child timed out` error as expected.
https://github.com/llvm/llvm-project/pull/94751
___
cfe-commits mailing list
cfe-commits
}}, 5.0f}; // expected-error
{{type 'float' cannot be narrowed to 'int' in initializer list}} //
expected-note {{insert an explicit cast to silence this issue}}
+}
``
https://github.com/llvm/llvm-project/pull/94752
___________
cfe-commits mailing list
cfe-co
GetProgramPath("nvptx-arch");
- auto StdoutOrErr = executeToolChainProgram(Program);
+ auto StdoutOrErr = executeToolChainProgram(Program, /*Timeout=*/10);
if (!StdoutOrErr)
return StdoutOrErr.takeError();
``
https://github.com/llvm/llvm-project/pull/94751
______
ote {{insert an explicit cast to silence this issue}}
+}
___________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
uto StdoutOrErr = executeToolChainProgram(Program, /*Timeout=*/10);
if (!StdoutOrErr)
return StdoutOrErr.takeError();
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
0.21.6.
- [Release notes](https://github.com/tianocore/edk2-pytool-library/releases)
-
[Commits](https://github.com/tianocore/edk2-pytool-library/compare/v0.21.5...v0.21.6)
---
updated-dependencies:
- dependency-name: edk2-pytool-library
dependency-type: direct:production
update-type: v
ter::Inner _3(t3, 42, s);
+Outer::Inner _4(t4, 42, s);
+Outer::Inner _5(t5, 42, s);
+Outer::Inner _6(t6, 42, s);
+Outer::Inner _7(t7, 42, s);
+Outer::Inner _8(t8, 42, s);
+Outer::Inner _9(t9, 42, 24, s);
+
+} // namespace GH94614
``
https://github.com/llvm/llvm-project/pull/94740
__
0.27.5 to 0.27.6.
- [Release notes](https://github.com/tianocore/edk2-pytool-extensions/releases)
-
[Commits](https://github.com/tianocore/edk2-pytool-extensions/compare/v0.27.5...v0.27.6)
---
updated-dependencies:
- dependency-name: edk2-pytool-extensions
dependency-type: direct:production
https://github.com/zyn0217 ready_for_review
https://github.com/llvm/llvm-project/pull/94740
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ttributes()` for ones that require late parsing?
CC @erichkeane in case he has opinions.
https://github.com/llvm/llvm-project/pull/67520
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
return true;
DC = DC->getParent();
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
DavidSpickett wrote:
Looks like this fixed things:
https://lab.llvm.org/buildbot/#/builders/245/builds/25578
Thanks for working with me on this.
https://github.com/llvm/llvm-project/pull/94603
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/tblah approved this pull request.
https://github.com/llvm/llvm-project/pull/92430
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ence_wrapper.
https://github.com/llvm/llvm-project/pull/94670
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/fberger approved this pull request.
https://github.com/llvm/llvm-project/pull/94362
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
the information is not
> lost to time, would suffice for now.
Originally these checks were using the same logic to determine whether the
variable was mutated. I don't know why why this was changed for one of these
checks.
https://github.com/llvm/llvm-project/pull/94362
______
-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/EricWF requested changes to this pull request.
Please see the previous comment for the requested changes.
https://github.com/llvm/llvm-project/pull/94670
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https
ilnik777 wrote:
The call function pointer is in the base class though. I always want the call
pointer and only sometimes the destructor pointer.
https://github.com/llvm/llvm-project/pull/94670
_______
llvm-branch-commits mailing list
llvm-branch-com
d, and should be removed.
It causes the optimizer to lose track of the `__call_` function pointer, which
prevents the ellision of the call. That's a big issue, and one that I think
should block.
Further, the use of `__small_buffer` prevents the ellision of the allocations,
further hurting the o
github.com/llvm/llvm-project/pull/94735
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
t6E2M3FN);
+}
arsenm wrote:
Probably would be useful as a helper somewhere in APFloat or Semantics
https://github.com/llvm/llvm-project/pull/94735
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/
steakhal wrote:
Let's do another round.
https://github.com/llvm/llvm-project/pull/93408
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/steakhal updated
https://github.com/llvm/llvm-project/pull/93408
>From f9e841ddaa865d529c806b2d115d5ddbc7109243 Mon Sep 17 00:00:00 2001
From: Balazs Benics
Date: Sun, 26 May 2024 11:40:01 +0200
Subject: [PATCH 01/16] [analyzer] Refine invalidation caused by `fread`
This
https://github.com/steakhal updated
https://github.com/llvm/llvm-project/pull/93408
>From f9e841ddaa865d529c806b2d115d5ddbc7109243 Mon Sep 17 00:00:00 2001
From: Balazs Benics
Date: Sun, 26 May 2024 11:40:01 +0200
Subject: [PATCH 01/16] [analyzer] Refine invalidation caused by `fread`
This
*fp = fopen("/home/test", "rb+");
+ if (fp) {
+// 3*5: 15 bytes read; which is not exactly 4 integers, thus we invalidate
the whole buffer.
steakhal wrote:
I can't dump that because one byte (the last one) of that 'int' object is left
uninitialized by t
-264,6 +267,7 @@ static void EmitDirectivesDecl(RecordKeeper ,
raw_ostream ) {
OS << "constexpr std::size_t getMaxLeafCount() { return "
<< GetMaxLeafCount(DirLang) << "; }\n";
OS << "Association getDirectiveAssociation(Directive D);\n";
+ OS << "Category getDirectiveCategory(Directive D);\n";
if (EnumHelperFuncs.length() > 0) {
OS << EnumHelperFuncs;
OS << "\n";
@@ -743,7 +747,29 @@ static void GenerateGetDirectiveAssociation(const
DirectiveLanguage ,
<< "::" << getAssocName(F->second) << ";\n";
}
}
- OS << " } // switch(Dir)\n";
+ OS << " } // switch (Dir)\n";
+ OS << " llvm_unreachable(\"Unexpected directive\");\n";
+ OS << "}\n";
+}
+
+static void GenerateGetDirectiveCategory(const DirectiveLanguage ,
+ raw_ostream ) {
+ std::string LangNamespace = "llvm::" + DirLang.getCppNamespace().str();
+ std::string CategoryTypeName = LangNamespace + "::Category";
+ std::string CategoryNamespace = CategoryTypeName + "::";
+
+ OS << '\n';
+ OS << CategoryTypeName << ' ' << LangNamespace << "::getDirectiveCategory("
+ << GetDirectiveType(DirLang) << " Dir) {\n";
+ OS << " switch (Dir) {\n";
+
+ for (Record *R : DirLang.getDirectives()) {
+Directive D{R};
+OS << " case " << GetDirectiveName(DirLang, R) << ":\n";
+OS << "return " << CategoryNamespace
+ << D.getCategory()->getValueAsString("name") << ";\n";
+ }
+ OS << " } // switch (Dir)\n";
OS << " llvm_unreachable(\"Unexpected directive\");\n";
OS << "}\n";
}
@@ -1196,6 +1222,9 @@ void EmitDirectivesBasicImpl(const DirectiveLanguage
,
// getDirectiveAssociation(Directive D)
GenerateGetDirectiveAssociation(DirLang, OS);
+ // getDirectiveCategory(Directive D)
+ GenerateGetDirectiveCategory(DirLang, OS);
+
// Leaf table for getLeafConstructs, etc.
EmitLeafTable(DirLang, OS, "LeafConstructTable");
}
>From 88403df321fe47145cd797388b352f3f934bab2b Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek
Date: Thu, 6 Jun 2024 17:11:17 -0500
Subject: [PATCH 2/3] typo
---
llvm/include/llvm/Frontend/Directive/DirectiveBase.td | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/llvm/include/llvm/Frontend/Directive/DirectiveBase.td
b/llvm/include/llvm/Frontend/Directive/DirectiveBase.td
index 6889c7e642ec6..707239c7ba364 100644
--- a/llvm/include/llvm/Frontend/Directive/DirectiveBase.td
+++ b/llvm/include/llvm/Frontend/Directive/DirectiveBase.td
@@ -203,6 +203,6 @@ class Directive {
// What the directive is associated with.
Association association = AS_FromLeaves;
- // The category if the directive.
+ // The category of the directive.
Category category = ?;
}
>From b243f852d7ac3fcc4cbaa3622ecaa9d7b5cde3ee Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek
Date: Fri, 7 Jun 2024 07:30:49 -0500
Subject: [PATCH 3/3] Update llvm/include/llvm/Frontend/OpenACC/ACC.td
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Co-authored-by: Valentin Clement (バレンタイン クレメン)
---
llvm/include/llvm/Frontend/OpenACC/ACC.td | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/llvm/include/llvm/Frontend/OpenACC/ACC.td
b/llvm/include/llvm/Frontend/OpenACC/ACC.td
index 30a81efe7f8a6..cda1d96b2a8cf 100644
--- a/llvm/include/llvm/Frontend/OpenACC/ACC.td
+++ b/llvm/include/llvm/Frontend/OpenACC/ACC.td
@@ -310,7 +310,7 @@ def ACC_Declare : Directive<"declare"> {
VersionedClause
];
let association = AS_None;
- let category = CA_Executable;
+ let category = CA_Declarative;
}
// 2.5.3
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/kovdan01 edited
https://github.com/llvm/llvm-project/pull/93904
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
shouldn't as the lvalue to rvalue conversion (aka. the load) operation
should have been done before the call.
In contrast to this, CSA appears to contradict this assumption, and the
`clang_analyzer_dump_char()` would have dumped `3 S8b`, which is the correct
solution for little-endian systems - like the one I pinned as the target
I fixed this by dumping these as chars, and refined the expectation messages to
meet this.
https://github.com/llvm/llvm-project/pull/93408
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ChuanqiXu9 wrote:
Thanks for testing. I've resent
https://github.com/llvm/llvm-project/commit/5a0181f568e56e37df80d0f74eca4775776fa8cd.
https://github.com/llvm/llvm-project/pull/92083
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https
+return 43;
+}
+}
+
+namespace A {
+using A_Impl::getAImpl;
+}
+
+export inline int getA() {
+return 43;
+}
+
+export inline int getA2(int) {
+return 88;
+}
+
+//--- m-partA.v1.cppm
+export module m:partA;
+
+namespace A_Impl {
+inline int getAImpl() {
+return 43;
+
// We have an 'int' binding at offset 0 of value 3.
+// We read 4 bytes at byte offset: 1,2,3,4.
+if (4 == fread(asChar + 1, 1, 4, fp)) {
+ clang_analyzer_dump(buffer[0]); // expected-warning{{3 S32b}} FIXME
Reading a 8 bit 'char' should not result in a 'S32b' value.
---
ChuanqiXu9 wrote:
Thank you very much!
https://github.com/llvm/llvm-project/pull/94603
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
__
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
are subscribed to the Google Groups
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to jenkinsci-commits+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/jenkinsci-commits
ON_CVREF
```
Now I still hate using the preprocessor, and dislike creating non-modular
re-includable headers. And I have a strong aversion to its use. But diagnostics
don't appear to be an issue here.
https://github.com/llvm/llvm-project/pull/94670
______
https://github.com/Michael137 approved this pull request.
LGTM, thanks for the tests
https://github.com/llvm/llvm-project/pull/94744
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
this message because you are subscribed to the Google Groups
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to jenkinsci-commits+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid
that
the functions should behave differently (and I'm not aware of any), it seems
better for them to keep using the same logic.
https://github.com/llvm/llvm-project/pull/94664
___________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
801 - 900 of 1864766 matches
Mail list logo
