nickdesaulniers wrote:
I think the libc buildbots are now failing as a result of this change. @aaupov
can you PTAL?
- https://lab.llvm.org/buildbot/#/builders/223/builds/34823
- https://lab.llvm.org/buildbot/#/builders/225/builds/29950
https://github.com/llvm/llvm-project/pull/69133
felix642 wrote:
@11happy I've compiled your code to test it on llvm's project and clang-tidy
seems to sometimes crash. Would you be able to have a look before we merge
this PR ?
```
PLEASE submit a bug report to https://github.com/llvm/llvm-project/issues/ and
include the crash backtrace.
efriedma-quic wrote:
I can sort of see how a define could be useful, but this doesn't match any
existing usage of __USER_LABEL_PREFIX__ given the relevant rules. I think
reusing the name is more likely to cause confusion, rather than help anyone.
https://github.com/cooperp approved this pull request.
Not code i'm familiar with, but looks like a mechanical change. LGTM
https://github.com/llvm/llvm-project/pull/79073
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/dpaoliello approved this pull request.
https://github.com/llvm/llvm-project/pull/79067
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
topperc wrote:
Please add "MC" to the title to distinquish from the other patch.
https://github.com/llvm/llvm-project/pull/78970
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Amir Ayupov
Date: 2024-01-22T14:47:42-08:00
New Revision: b83b8d3fd17885438b0ea154e07088d877d293a8
URL:
https://github.com/llvm/llvm-project/commit/b83b8d3fd17885438b0ea154e07088d877d293a8
DIFF:
https://github.com/llvm/llvm-project/commit/b83b8d3fd17885438b0ea154e07088d877d293a8.diff
@@ -100,6 +100,8 @@ on support follow.
``V``Supported
``Za128rs`` Supported (`See note
<#riscv-profiles-extensions-note>`__)
``Za64rs`` Supported (`See note
<#riscv-profiles-extensions-note>`__)
+ ``Zaamo``Supported
@@ -0,0 +1,11 @@
+# RUN: not llvm-mc -triple riscv32 -mattr=+experimental-zaamo < %s 2>&1 |
FileCheck %s
topperc wrote:
Is this duplicating existing A test cases? If so can we split the A extension
tests into 2 files as a pre-commit and add these the new Zaamo
@@ -1105,7 +1129,7 @@ def FeatureForcedAtomics : SubtargetFeature<
"forced-atomics", "HasForcedAtomics", "true",
"Assume that lock-free native-width atomics are available">;
def HasAtomicLdSt
-: Predicate<"Subtarget->hasStdExtA() || Subtarget->hasForcedAtomics()">;
https://github.com/topperc edited
https://github.com/llvm/llvm-project/pull/78970
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/topperc requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/78970
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/66915
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ldionne demilestoned
https://github.com/llvm/llvm-project/pull/77176
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -360,11 +365,13 @@ _LIBCPP_HARDENING_MODE_DEBUG
#define _LIBCPP_ASSERT_VALID_ELEMENT_ACCESS(expression, message)
_LIBCPP_ASSERT(expression, message)
#define _LIBCPP_ASSERT_NON_NULL(expression, message)
_LIBCPP_ASSERT(expression, message)
#
@@ -52,204 +51,234 @@
class ComparisonResults {
ldionne wrote:
We need to take the parts of this test that would reproduce an OOB access
inside `std::sort` & friends to a new test file. And that one should be enabled
whenever the valid-element-access checks
https://github.com/ilovepi edited
https://github.com/llvm/llvm-project/pull/79061
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-backend-x86
Author: Paul Kirth (ilovepi)
Changes
Currently, the UnifiedLTO pipeline seems to have trouble with several
LTO features, like SplitLTO units, which means we cannot use important
optimizations like Whole Program Devirtualization or security
https://github.com/ilovepi created
https://github.com/llvm/llvm-project/pull/79061
Currently, the UnifiedLTO pipeline seems to have trouble with several
LTO features, like SplitLTO units, which means we cannot use important
optimizations like Whole Program Devirtualization or security hardening
https://github.com/seranu edited https://github.com/llvm/llvm-project/pull/78957
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jplehr closed https://github.com/llvm/llvm-project/pull/76571
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Jan Patrick Lehr
Date: 2024-01-22T21:59:26+01:00
New Revision: fa4780fa6cc36188b84b2a977ac15351c39d45dd
URL:
https://github.com/llvm/llvm-project/commit/fa4780fa6cc36188b84b2a977ac15351c39d45dd
DIFF:
@@ -283,9 +283,20 @@
// - `_LIBCPP_ASSERT_COMPATIBLE_ALLOCATOR` -- checks any operations that
exchange nodes between containers to make sure
// the containers have compatible allocators.
//
+// - `_LIBCPP_ASSERT_ARGUMENT_WITHIN_DOMAIN` -- checks that the given argument
is
seanm wrote:
@bwendling thanks for your reply. No idea how representative it is of other
projects, but 3 of the 6 flexible arrays in
[libusb](https://github.com/libusb/libusb) structures have these kinds of
non-trivial counts.
The GCC folks seem to have
ilovepi wrote:
@topperc Is there anything else you think needs to be done for the code
generation?
https://github.com/llvm/llvm-project/pull/66915
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -0,0 +1,151 @@
+//===--- UseStdMinMaxCheck.cpp - clang-tidy
---===//
+//
+// 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:
@@ -0,0 +1,151 @@
+//===--- UseStdMinMaxCheck.cpp - clang-tidy
---===//
+//
+// 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.
Except reported nits, looks fine.
https://github.com/llvm/llvm-project/pull/77816
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -0,0 +1,151 @@
+//===--- UseStdMinMaxCheck.cpp - clang-tidy
---===//
+//
+// 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 edited
https://github.com/llvm/llvm-project/pull/77816
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,151 @@
+//===--- UseStdMinMaxCheck.cpp - clang-tidy
---===//
+//
+// 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:
@@ -0,0 +1,151 @@
+//===--- UseStdMinMaxCheck.cpp - clang-tidy
---===//
+//
+// 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:
@@ -0,0 +1,151 @@
+//===--- UseStdMinMaxCheck.cpp - clang-tidy
---===//
+//
+// 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:
Artem-B wrote:
> * Address space cast of nullptr in local_space into a generic_space for the
> CUDA backend.
I think you mean "NVPTX back-end". CUDA is a front-end entity (C++ w/ GPU
extensions)
https://github.com/llvm/llvm-project/pull/78759
___
https://github.com/5chmidti closed
https://github.com/llvm/llvm-project/pull/78320
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Julian Schmidt
Date: 2024-01-22T21:51:39+01:00
New Revision: 9f8ccf50dde17adae1368a0cf41edadc8327aaf4
URL:
https://github.com/llvm/llvm-project/commit/9f8ccf50dde17adae1368a0cf41edadc8327aaf4
DIFF:
https://github.com/5chmidti edited
https://github.com/llvm/llvm-project/pull/78320
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -285,6 +289,20 @@ void
NVPTXTargetCodeGenInfo::addNVVMMetadata(llvm::GlobalValue *GV,
bool NVPTXTargetCodeGenInfo::shouldEmitStaticExternCAliases() const {
return false;
}
+
+llvm::Constant *
+NVPTXTargetCodeGenInfo::getNullPointer(const CodeGen::CodeGenModule ,
+
https://github.com/zahiraam closed
https://github.com/llvm/llvm-project/pull/78330
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Zahira Ammarguellat
Date: 2024-01-22T15:50:24-05:00
New Revision: 364a5b5b850a4f3e564da4bde080567a5d1e14d2
URL:
https://github.com/llvm/llvm-project/commit/364a5b5b850a4f3e564da4bde080567a5d1e14d2
DIFF:
https://github.com/zahiraam edited
https://github.com/llvm/llvm-project/pull/78330
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -547,18 +547,20 @@ static void PrintVersion(raw_ostream ) {
// Dump the configuration.
static int dumpConfig(bool IsSTDIN) {
std::unique_ptr Code;
- // We can't read the code to detect the language if there's no file name.
- if (!IsSTDIN) {
-// Read in the code in
https://github.com/bhamiltoncx updated
https://github.com/llvm/llvm-project/pull/79051
>From 893c295ea7ff5fda9953420068fae2af7457c5c1 Mon Sep 17 00:00:00 2001
From: Ben Hamilton
Date: Mon, 22 Jan 2024 11:33:18 -0700
Subject: [PATCH] [Format] Fix detection of languages when reading from stdin
https://github.com/gulfemsavrun updated
https://github.com/llvm/llvm-project/pull/79051
>From 893c295ea7ff5fda9953420068fae2af7457c5c1 Mon Sep 17 00:00:00 2001
From: Ben Hamilton
Date: Mon, 22 Jan 2024 11:33:18 -0700
Subject: [PATCH] [Format] Fix detection of languages when reading from stdin
@@ -0,0 +1,5 @@
+// RUN: clang-format -dump-config - < %s | FileCheck %s
bhamiltoncx wrote:
In this case, we do want two tests (at least, we do need the integration test
in `clang/test/Format/dump-config-objc-stdin.m` to reproduce the issue).
The reason is,
bwendling wrote:
> @bwendling is there any plan / possibility for simple expressions (with no
> side effects)? Like:
>
> ```c
> struct libusb_bos_dev_capability_descriptor {
> uint8_t bLength;
> uint8_t bDescriptorType;
> uint8_t bDevCapabilityType;
> uint8_t
https://github.com/shiltian updated
https://github.com/llvm/llvm-project/pull/78333
>From 2fb901b2379290b162647f0b6d152b93df1b1e7f Mon Sep 17 00:00:00 2001
From: Shilei Tian
Date: Mon, 22 Jan 2024 15:39:19 -0500
Subject: [PATCH] [Clang][Driver] Fix `--save-temps` for OpenCL AoT compilation
We
HighCommander4 wrote:
> couldn't we just use the .clang-format-ignore file to say... hey don't bother
> with this file, we can't do it...
I think that's a good solution for avoiding the OOM when actually trying to
format the file, in combination with this patch for avoiding the OOM in
@@ -53,10 +53,9 @@ struct __debug_less {
_LIBCPP_CONSTEXPR_SINCE_CXX14 inline _LIBCPP_HIDE_FROM_ABI void
__do_compare_assert(long, _LHS&, _RHS&) {}
};
-// Pass the comparator by lvalue reference. Or in debug mode, using a
-// debugging wrapper that stores a reference.
-#if
@@ -0,0 +1,5 @@
+// RUN: clang-format -dump-config - < %s | FileCheck %s
HazardyKnusperkeks wrote:
I don't think you need this, right? The unit tests does the same, or not?
https://github.com/llvm/llvm-project/pull/79051
@@ -547,18 +547,20 @@ static void PrintVersion(raw_ostream ) {
// Dump the configuration.
static int dumpConfig(bool IsSTDIN) {
std::unique_ptr Code;
- // We can't read the code to detect the language if there's no file name.
- if (!IsSTDIN) {
-// Read in the code in
@@ -135,14 +135,14 @@ void
Thumb1InstrInfo::loadRegFromStackSlot(MachineBasicBlock ,
void Thumb1InstrInfo::expandLoadStackGuard(
MachineBasicBlock::iterator MI) const {
MachineFunction = *MI->getParent()->getParent();
- const TargetMachine = MF.getTarget();
const
@@ -544,28 +544,23 @@ static void PrintVersion(raw_ostream ) {
}
// Dump the configuration.
-static int dumpConfig() {
- StringRef FileName;
+static int dumpConfig(bool IsSTDIN) {
std::unique_ptr Code;
- if (FileNames.empty()) {
-// We can't read the code to detect
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Ben Hamilton (Ben Gertzfield) (bhamiltoncx)
Changes
The code cleanup in #74794 accidentally broke detection of languages by
reading file content from stdin, e.g. via `clang-format -dump-config -
/path/to/filename`.
This PR adds
https://github.com/ldionne edited
https://github.com/llvm/llvm-project/pull/77405
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bhamiltoncx created
https://github.com/llvm/llvm-project/pull/79051
The code cleanup in #74794 accidentally broke detection of languages by reading
file content from stdin, e.g. via `clang-format -dump-config - <
/path/to/filename`.
This PR adds unit and integration tests
@@ -0,0 +1,159 @@
+//===--- IncludesSeparator.cpp ---*- C++ -*-===//
HazardyKnusperkeks wrote:
Does this have to be done in a separate pass? Couldn't it be done in the normal
formatting?
https://github.com/llvm/llvm-project/pull/78957
@@ -4831,6 +4848,7 @@ struct FormatStyle {
DerivePointerAlignment == R.DerivePointerAlignment &&
DisableFormat == R.DisableFormat &&
EmptyLineAfterAccessModifier == R.EmptyLineAfterAccessModifier &&
+ EmptyLinesAfterIncludes ==
carlos4242 wrote:
@owenca ... is this better for you?
https://github.com/llvm/llvm-project/pull/79037
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HighCommander4 wrote:
> Can we write a unit test for this?
I'm open to trying but there are a few open questions in my mind:
* Is it fine if the failure mode of a testcase is triggering an OOM?
* Triggering the problematic codepath requires a combination of a large file
(thousands of lines)
@@ -3220,6 +3220,25 @@ the configuration (without a prefix: ``Auto``).
+.. _EmptyLinesAfterIncludes:
+
+**EmptyLinesAfterIncludes** (``Unsigned``) :versionbadge:`clang-format 18`
:ref:`¶ `
HazardyKnusperkeks wrote:
Where do I have to sign up for that
mmoadeli wrote:
> It would be great to add some tests for local AS null pointers for NVPTX and
> AMDGPU back-ends.
They already have it
[here](https://github.com/llvm/llvm-project/blob/286ac8f3ea6aec711827ccab9608b010e78b18cf/clang/test/CodeGenSYCL/amd-address-space-conversions.cpp#L24)
https://github.com/ldionne approved this pull request.
@philnik777
> Does this actually improve the performance anywhere? AFAICT all we gain is
> avoiding the messages being generated in the binary, but I'm not convinced
> that's much of a problem. e.g. in the `std::next` case, there doesn't
@@ -350,6 +363,9 @@ _LIBCPP_HARDENING_MODE_DEBUG
#define _LIBCPP_ASSERT_PEDANTIC(expression, message)
_LIBCPP_ASSUME(expression)
#define _LIBCPP_ASSERT_INTERNAL(expression, message)
_LIBCPP_ASSUME(expression)
#define
https://github.com/ldionne edited
https://github.com/llvm/llvm-project/pull/77176
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mydeveloperday wrote:
While I get this fixes the guessLanguage limitation, if its OOM the I guess
isn't it going to be OOM for any format? (as you mention above), couldn't we
just use the .clang-format-ignore file to say... hey don't bother with this
file, we can't do it...
I feel like this
Mirko =?utf-8?q?Brkušanin?= ,
Mirko =?utf-8?q?Brkušanin?=
Message-ID:
In-Reply-To:
@@ -305,6 +305,11 @@ class VOP3OpSel_gfx10 op, VOPProfile p> :
VOP3e_gfx10 {
class VOP3OpSel_gfx11_gfx12 op, VOPProfile p> : VOP3OpSel_gfx10;
+class VOP3FP8OpSel_gfx11_gfx12 op, VOPProfile
pogo59 wrote:
I note that these macros are just name substitutions, unlike the function-style
macros in ia32intrin.h. I didn't change the definitions.
https://github.com/llvm/llvm-project/pull/79048
___
cfe-commits mailing list
@@ -243,6 +247,14 @@ void UnwrappedLineParser::parse() {
Callback.consumeUnwrappedLine(Line);
}
Callback.finishRun();
+TotalLinesProcessed += Lines.size();
+if ((TotalLinesProcessed / 1) > LastReport) {
HighCommander4 wrote:
My bad,
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/78925
>From 47ebe97c199ade54a9538eaed02171d78e6a5a46 Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Sun, 21 Jan 2024 02:45:16 -0500
Subject: [PATCH] [clang-format] Limit how much work guessLanguage() can do
https://github.com/ilovepi approved this pull request.
Thanks for the patch. This is a great catch!
https://github.com/llvm/llvm-project/pull/79041
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/78991
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: cor3ntin
Date: 2024-01-22T21:19:29+01:00
New Revision: 02a28ee8d981b2a8416c50e41c74cb21e73b8340
URL:
https://github.com/llvm/llvm-project/commit/02a28ee8d981b2a8416c50e41c74cb21e73b8340
DIFF:
https://github.com/llvm/llvm-project/commit/02a28ee8d981b2a8416c50e41c74cb21e73b8340.diff
https://github.com/HazardyKnusperkeks approved this pull request.
https://github.com/llvm/llvm-project/pull/78921
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mydeveloperday commented:
Can we write a unit test for this?
https://github.com/llvm/llvm-project/pull/78925
___
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
@llvm/pr-subscribers-backend-x86
Author: Paul T Robinson (pogo59)
Changes
These are largely copy-pasted from the corresponding function descriptions.
Added \see cross-references. Also changed c tags to \c.
---
Full diff:
https://github.com/asb approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/79030
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HazardyKnusperkeks approved this pull request.
https://github.com/llvm/llvm-project/pull/78847
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/pogo59 created
https://github.com/llvm/llvm-project/pull/79048
These are largely copy-pasted from the corresponding function descriptions.
Added \see cross-references. Also changed tags to \c.
>From 5a3f36a9276e0919d72cb48a7d30c2951b542c79 Mon Sep 17 00:00:00 2001
From:
@@ -243,6 +247,14 @@ void UnwrappedLineParser::parse() {
Callback.consumeUnwrappedLine(Line);
}
Callback.finishRun();
+TotalLinesProcessed += Lines.size();
+if ((TotalLinesProcessed / 1) > LastReport) {
mydeveloperday wrote:
Do we
https://github.com/HazardyKnusperkeks approved this pull request.
https://github.com/llvm/llvm-project/pull/78752
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ldionne approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/77164
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ributzka updated
https://github.com/llvm/llvm-project/pull/78816
>From 968941561e08424ecb4fb15615ee46fe8356405e Mon Sep 17 00:00:00 2001
From: Juergen Ributzka
Date: Fri, 19 Jan 2024 16:10:03 -0800
Subject: [PATCH 1/2] [clang][modules] Fix CodeGen options that can affect the
https://github.com/carlos4242 updated
https://github.com/llvm/llvm-project/pull/79037
>From 61a06b77e8e6f2b5c5a9a2aa0b7f46260545f5b4 Mon Sep 17 00:00:00 2001
From: Carl Peto
Date: Mon, 22 Jan 2024 18:52:46 +
Subject: [PATCH] [clang] - Sema::isSimpleTypeSpecifier return true for _Bool
in
@@ -3220,6 +3220,25 @@ the configuration (without a prefix: ``Auto``).
+.. _EmptyLinesAfterIncludes:
+
+**EmptyLinesAfterIncludes** (``Unsigned``) :versionbadge:`clang-format 18`
:ref:`¶ `
mydeveloperday wrote:
18 will be correct today, but it won't be
@@ -7007,27 +7090,69 @@ static FloatingRank getFloatingRank(QualType T) {
}
}
+/// C++23 6.8.5 [conv.rank]
/// getFloatingTypeOrder - Compare the rank of the two specified floating
/// point types, ignoring the domain of the type (i.e. 'double' ==
-/// '_Complex double').
https://github.com/mydeveloperday edited
https://github.com/llvm/llvm-project/pull/78996
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -276,13 +276,44 @@ void FormatTokenLexer::tryMergePreviousTokens() {
return;
}
}
- // TableGen's Multi line string starts with [{
- if (Style.isTableGen() && tryMergeTokens({tok::l_square, tok::l_brace},
-
@@ -276,13 +276,44 @@ void FormatTokenLexer::tryMergePreviousTokens() {
return;
}
}
- // TableGen's Multi line string starts with [{
- if (Style.isTableGen() && tryMergeTokens({tok::l_square, tok::l_brace},
-
https://github.com/mydeveloperday commented:
I don't use TableGen myself so I can't really say if this is quite the correct
thing to do, I feel it needs a TableGenTest.cpp example of real TableGen code
for every example you give here. Then we should try and cover the other use
cases. That we
@@ -112,6 +112,85 @@ enum FloatingRank {
Ibm128Rank
};
+constexpr unsigned CXX23FloatRankToIndex(clang::BuiltinType::Kind Kind) {
+ switch (Kind) {
+ case clang::BuiltinType::Float16:
+return 0;
+ case clang::BuiltinType::BFloat16:
+return 1;
+ case
@@ -401,6 +401,8 @@ static bool initTargetOptions(DiagnosticsEngine ,
Options.UniqueBasicBlockSectionNames =
CodeGenOpts.UniqueBasicBlockSectionNames;
Options.TLSSize = CodeGenOpts.TLSSize;
+ // TODO: Add correct codegen options in LLVM
+ // Options.TLSDesc =
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/79041
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jakeegan wrote:
@jansvoboda11 Thanks again for investigating!
https://github.com/llvm/llvm-project/pull/78748
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/qiongsiwu closed
https://github.com/llvm/llvm-project/pull/78285
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Qiongsi Wu
Date: 2024-01-22T14:54:58-05:00
New Revision: 4f21fb84479286ddc781d73f8df152f81a8264e7
URL:
https://github.com/llvm/llvm-project/commit/4f21fb84479286ddc781d73f8df152f81a8264e7
DIFF:
https://github.com/llvm/llvm-project/commit/4f21fb84479286ddc781d73f8df152f81a8264e7.diff
Author: Qiongsi Wu
Date: 2024-01-22T14:54:58-05:00
New Revision: 4f21fb84479286ddc781d73f8df152f81a8264e7
URL:
https://github.com/llvm/llvm-project/commit/4f21fb84479286ddc781d73f8df152f81a8264e7
DIFF:
https://github.com/llvm/llvm-project/commit/4f21fb84479286ddc781d73f8df152f81a8264e7.diff
qiongsiwu wrote:
Hi @vitalybuka ! I am landing this PR. The precommit CI failures do not seem to
related to changes in this PR. Please let me know if you see failures and I
will revert. Thanks so much!
https://github.com/llvm/llvm-project/pull/78285
https://github.com/jakeegan closed
https://github.com/llvm/llvm-project/pull/78748
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Jake Egan
Date: 2024-01-22T14:53:35-05:00
New Revision: 02aa6956b5b6c92f3d4aea0aa466ed0d33343335
URL:
https://github.com/llvm/llvm-project/commit/02aa6956b5b6c92f3d4aea0aa466ed0d33343335
DIFF:
https://github.com/llvm/llvm-project/commit/02aa6956b5b6c92f3d4aea0aa466ed0d33343335.diff
101 - 200 of 673 matches
Mail list logo