https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/71410
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman approved this pull request.
LGTM, sorry for missing that during previous review!
https://github.com/llvm/llvm-project/pull/71410
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mail
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/71410
>From 8984ebb39a4bb2ba475440b0ed41c534f586a4c6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Mon, 6 Nov 2023 16:59:33 +0100
Subject: [PATCH] [clang][Interp] Fix creating APSInt from Integral
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 8f76522a61d01cf7d70debd39418259e969bb8d6
81836da057f2703f49f30d31955306577a0e1170 --
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
The boolean argument in the APSInt constructor is IsUnsigned, not IsSigned.
This marks the 10th time I've run into this issue. Happy anniversary.
---
Full diff: https://github.com/llvm/llvm-project/pull/7141
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/71410
The boolean argument in the APSInt constructor is IsUnsigned, not IsSigned.
This marks the 10th time I've run into this issue. Happy anniversary.
>From 81836da057f2703f49f30d31955306577a0e1170 Mon Sep 17 00:00: