https://github.com/steakhal closed
https://github.com/llvm/llvm-project/pull/107537
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
NagyDonat wrote:
> LGTM. FYI "modelled" should contain only 1 "l" if I'm not mistaken.
British English uses "modeled", while American uses "modelled". I don't know
which is preferred in LLVM.
https://github.com/llvm/llvm-project/pull/107537
___
cfe-
necto wrote:
> LGTM. FYI "modelled" should contain only 1 "l" if I'm not mistaken. Also llvm
> style suggests capitalizing and punctuating comments. None of these are
> blockers.
fixed eecf42ac8fc4
https://github.com/llvm/llvm-project/pull/107537
__
https://github.com/necto updated
https://github.com/llvm/llvm-project/pull/107537
>From 63c856732aeda977786534d66597d0aba12cb0d4 Mon Sep 17 00:00:00 2001
From: Arseniy Zaostrovnykh
Date: Tue, 3 Sep 2024 18:01:02 +0200
Subject: [PATCH 1/2] [analyzer] Model constructor initializer for an array
m
https://github.com/steakhal approved this pull request.
LGTM. FYI "modelled" should contain only 1 "l" if I'm not mistaken. Also llvm
style suggests capitalizing and punctuating comments. None of these are
blockers.
https://github.com/llvm/llvm-project/pull/107537
_
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Arseniy Zaostrovnykh (necto)
Changes
Bind the array member to the compound region associated with the initializer
list, e.g.:
class C {
int arr[2];
C() : arr{1, 2} {}
};
C c;
This change enables
https://github.com/necto created
https://github.com/llvm/llvm-project/pull/107537
Bind the array member to the compound region associated with the initializer
list, e.g.:
class C {
int arr[2];
C() : arr{1, 2} {}
};
C c;
This change enables correct values in `c.arr[0]`
