[clang] [clang] Add fix for GH issue 83050 (PR #83071)

2024-02-26 Thread via cfe-commits
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

[clang] [clang] Add fix for GH issue 83050 (PR #83071)

2024-02-26 Thread via 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 +static constexpr unsigned short _Fract SQRT_F

[clang] [clang] Add fix for GH issue 83050 (PR #83071)

2024-02-26 Thread Nick Desaulniers via 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 +static constexpr unsigned short _Fract SQRT_F

[clang] [clang] Add fix for GH issue 83050 (PR #83071)

2024-02-26 Thread Nick Desaulniers via cfe-commits
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

[clang] [clang] Add fix for GH issue 83050 (PR #83071)

2024-02-26 Thread via cfe-commits
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

[clang] [clang] Add fix for GH issue 83050 (PR #83071)

2024-02-26 Thread via 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 +static constexpr unsigned short _Fract SQRT_F

[clang] [clang] Add fix for GH issue 83050 (PR #83071)

2024-02-26 Thread Nick Desaulniers via 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 +static constexpr unsigned short _Fract SQRT_F

[clang] [clang] Add fix for GH issue 83050 (PR #83071)

2024-02-26 Thread via cfe-commits
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

[clang] [clang] Add fix for GH issue 83050 (PR #83071)

2024-02-26 Thread Nick Desaulniers via 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

[clang] [clang] Add fix for GH issue 83050 (PR #83071)

2024-02-26 Thread via cfe-commits
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

[clang] [clang] Add fix for GH issue 83050 (PR #83071)

2024-02-26 Thread via cfe-commits
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