tbaederr wrote:
https://reviews.llvm.org/D156565 hasn't been merged as far as I can tell and
it's the only test case I have for this. :(
https://github.com/llvm/llvm-project/pull/67990
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://li
AaronBallman wrote:
> https://reviews.llvm.org/D156565 hasn't been merged as far as I can tell and
> it's the only test case I have for this. :(
Ah shoot, let me ping that review and see if I can get it accepted and landed
so we can have test coverage.
https://github.com/llvm/llvm-project/pul
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/67990
>From 0056d67f145fd0d5f8056325bc70246314c9b117 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Mon, 2 Oct 2023 16:26:46 +0200
Subject: [PATCH] [clang][Interp] Only emit function_param_value_un
https://github.com/AaronBallman approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/67990
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1,5 +1,5 @@
// RUN: %clang_cc1 -triple x86_64-apple-darwin10.0.0 -fsyntax-only -verify %s
-Winvalid-offsetof -std=c++98
-// RUN: %clang_cc1 -triple x86_64-apple-darwin10.0.0 -fsyntax-only
-verify=expected,new-interp %s -Winvalid-offsetof -std=c++98
-fexperimental-new-const
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/67990
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/67990
>From 0056d67f145fd0d5f8056325bc70246314c9b117 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Mon, 2 Oct 2023 16:26:46 +0200
Subject: [PATCH
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/67990
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/67990
This is also what the current interpreter does.
No test for now since the only reproducer I have is from
https://reviews.llvm.org/D156565 and doesn't work without that patch.
>From 5e43ab378e64ea8a852f8bf5571d
llvmbot wrote:
@llvm/pr-subscribers-clang
Changes
This is also what the current interpreter does.
No test for now since the only reproducer I have is from
https://reviews.llvm.org/D156565 and doesn't work without that patch.
---
Full diff: https://github.com/llvm/llvm-project/pull/67990.
tbaederr wrote:
Ping
https://github.com/llvm/llvm-project/pull/67990
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman commented:
Test coverage?
https://github.com/llvm/llvm-project/pull/67990
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
12 matches
Mail list logo