https://github.com/PiJoules updated
https://github.com/llvm/llvm-project/pull/83071
>From 306394b5e0386701f577ec2aa9a021492fbae734 Mon Sep 17 00:00:00 2001
From: Leonard Chan
Date: Mon, 26 Feb 2024 14:11:45 -0800
Subject: [PATCH] [clang] Fixes inf loop parsing fixed point literal
Clang was inc
@@ -0,0 +1,11 @@
+/// This is the regression test for
https://github.com/llvm/llvm-project/issues/83050.
+/// This just needs to compile.
+// RUN: %clang_cc1 -x c++ -ffixed-point -S %s -o /dev/null
-triple=x86_64-linux -ffixed-point
+static constexpr unsigned short _Fract SQRT_F
@@ -0,0 +1,11 @@
+/// This is the regression test for
https://github.com/llvm/llvm-project/issues/83050.
+/// This just needs to compile.
+// RUN: %clang_cc1 -x c++ -ffixed-point -S %s -o /dev/null
-triple=x86_64-linux -ffixed-point
+static constexpr unsigned short _Fract SQRT_F
nickdesaulniers wrote:
For the commit message and PR description, please consider:
1. using a subject that describes more of what the patch does, such as "[clang]
fixes inf loop parsing fixed point literal"
2. put "Fixes #83050" in the commit body; when merged, github will auto close
the corres
https://github.com/PiJoules updated
https://github.com/llvm/llvm-project/pull/83071
>From f14ff8098b748944cdc10bc1421c9002dd6ef16e Mon Sep 17 00:00:00 2001
From: Leonard Chan
Date: Mon, 26 Feb 2024 14:11:45 -0800
Subject: [PATCH] [clang] Add fix for GH issue 83050
Clang was incorrectly finding
@@ -0,0 +1,11 @@
+/// This is the regression test for
https://github.com/llvm/llvm-project/issues/83050.
+/// This just needs to compile.
+// RUN: %clang_cc1 -x c++ -ffixed-point -S %s -o /dev/null
-triple=x86_64-linux -ffixed-point
+static constexpr unsigned short _Fract SQRT_F
@@ -0,0 +1,11 @@
+/// This is the regression test for
https://github.com/llvm/llvm-project/issues/83050.
+/// This just needs to compile.
+// RUN: %clang_cc1 -x c++ -ffixed-point -S %s -o /dev/null
-triple=x86_64-linux -ffixed-point
+static constexpr unsigned short _Fract SQRT_F
https://github.com/lntue approved this pull request.
https://github.com/llvm/llvm-project/pull/83071
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,11 @@
+/// This is the regression test for
https://github.com/llvm/llvm-project/issues/83050.
+/// This just needs to compile.
+// RUN: %clang_cc1 -x c++ -ffixed-point -S %s -o /dev/null
-triple=x86_64-linux -ffixed-point
nickdesaulniers wrote:
Consi
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (PiJoules)
Changes
Clang was incorrectly finding the start of the exponent in a fixed point hex
literal. It would unconditionally find the first `e/E/p/P` in a constant
regardless of if it were hex or not and parser the remaining dig
https://github.com/PiJoules created
https://github.com/llvm/llvm-project/pull/83071
Clang was incorrectly finding the start of the exponent in a fixed point hex
literal. It would unconditionally find the first `e/E/p/P` in a constant
regardless of if it were hex or not and parser the remaining
11 matches
Mail list logo