compnerd wrote:
New version also includes the release notes for this.
https://github.com/llvm/llvm-project/pull/130993
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/compnerd updated
https://github.com/llvm/llvm-project/pull/130993
>From 1021d05a73cb990bcbdd1948fd372fdf4b1a21ec Mon Sep 17 00:00:00 2001
From: Saleem Abdulrasool
Date: Wed, 12 Mar 2025 09:52:58 -0700
Subject: [PATCH] Lex: add support for `i128` and `ui128` suffixes
Microsof
https://github.com/Fznamznon commented:
Perhaps a release note makes sense as well.
https://github.com/llvm/llvm-project/pull/130993
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zmodem approved this pull request.
> We use existing support for the Microsoft extension type `__int128` which has
> been there for a long time. I feel like we should have appropriate coverage
> already
Sounds good to me.
lgtm with the i128 test that Fznamznon suggested, an
https://github.com/zmodem commented:
> Take the opportunity to tighten up the code slightly by ensuring that we do
> not access out-of-bounds characters when lexing the token.
That's a nice touch :-)
Do we already have tests somewhere checking that we can codegen this, or do we
need to add th
compnerd wrote:
> > Take the opportunity to tighten up the code slightly by ensuring that we do
> > not access out-of-bounds characters when lexing the token.
>
> That's a nice touch :-)
😄
> Do we already have tests somewhere checking that we can codegen this, or do
> we need to add that?
I
@@ -3924,10 +3924,18 @@ ExprResult Sema::ActOnNumericConstant(const Token &Tok,
Scope *UDLScope) {
// to get the integer value from an overly-wide APInt is *extremely*
// expensive, so the naive approach of assuming
// llvm::IntegerType::MAX_INT_BITS is a big perfo
@@ -13,16 +13,24 @@ __int64 w = 0x43ui64;
__int64 z = 9Li64; // expected-error {{invalid suffix}}
__int64 q = 10lli64; // expected-error {{invalid suffix}}
-__complex double c1 = 1i;
-__complex double c2 = 1.0i;
+__complex double c1 = 1i; // GNU extension
+__complex double c
@@ -3924,10 +3924,18 @@ ExprResult Sema::ActOnNumericConstant(const Token &Tok,
Scope *UDLScope) {
// to get the integer value from an overly-wide APInt is *extremely*
// expensive, so the naive approach of assuming
// llvm::IntegerType::MAX_INT_BITS is a big perfo
@@ -13,16 +13,24 @@ __int64 w = 0x43ui64;
__int64 z = 9Li64; // expected-error {{invalid suffix}}
__int64 q = 10lli64; // expected-error {{invalid suffix}}
-__complex double c1 = 1i;
-__complex double c2 = 1.0i;
+__complex double c1 = 1i; // GNU extension
+__complex double c
https://github.com/Fznamznon edited
https://github.com/llvm/llvm-project/pull/130993
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
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 ab22f652a4dfcaf5b6884a85e498d7ca077937ca
ba82162bbc75ab6838f2c1aa0ad89cfe585a578f --e
https://github.com/compnerd updated
https://github.com/llvm/llvm-project/pull/130993
>From ba82162bbc75ab6838f2c1aa0ad89cfe585a578f Mon Sep 17 00:00:00 2001
From: Saleem Abdulrasool
Date: Wed, 12 Mar 2025 09:52:58 -0700
Subject: [PATCH] Lex: add support for `i128` and `ui128` suffixes
Microsof
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Saleem Abdulrasool (compnerd)
Changes
Microsoft's compiler supports an extension for 128-bit literals. This is
referenced in `intsafe.h` which is included transitievly. When building with
modules, the literal parsing causes a failure due
https://github.com/compnerd created
https://github.com/llvm/llvm-project/pull/130993
Microsoft's compiler supports an extension for 128-bit literals. This is
referenced in `intsafe.h` which is included transitievly. When building with
modules, the literal parsing causes a failure due to the mi
15 matches
Mail list logo
