diff options
author | Ilya Biryukov <ibiryukov@google.com> | 2018-07-23 06:32:36 +0000 |
---|---|---|
committer | Ilya Biryukov <ibiryukov@google.com> | 2018-07-23 06:32:36 +0000 |
commit | e7cd2c38e81e9916edfea6c3f07418b26518cb00 (patch) | |
tree | da5d2819c21966ecead52683013642d175c7999e /clang/include | |
parent | 781ee0bc5abade62a306873aa8fb47ea4d75fdf2 (diff) | |
download | bcm5719-llvm-e7cd2c38e81e9916edfea6c3f07418b26518cb00.tar.gz bcm5719-llvm-e7cd2c38e81e9916edfea6c3f07418b26518cb00.zip |
Revert "Fold dangling-field warning into general initialization lifetime checks."
This reverts commit r337627.
After the change, clang started producing invalid warning on the following code:
struct foo {
foo(char *x) : x_(&x[10]) {}
private:
char *x_;
};
1.cpp:2:21: warning: initializing pointer member 'x_' with the stack address of parameter 'x' [-Wdangling-field]
llvm-svn: 337671
Diffstat (limited to 'clang/include')
-rw-r--r-- | clang/include/clang/Basic/DiagnosticSemaKinds.td | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/clang/include/clang/Basic/DiagnosticSemaKinds.td b/clang/include/clang/Basic/DiagnosticSemaKinds.td index e45f5f3376a..5ae749fa933 100644 --- a/clang/include/clang/Basic/DiagnosticSemaKinds.td +++ b/clang/include/clang/Basic/DiagnosticSemaKinds.td @@ -7870,11 +7870,11 @@ def note_ref_var_local_bind : Note< // Check for initializing a member variable with the address or a reference to // a constructor parameter. def warn_bind_ref_member_to_parameter : Warning< - "binding reference member %0 to stack allocated " - "%select{variable|parameter}2 %1">, InGroup<DanglingField>; + "binding reference member %0 to stack allocated parameter %1">, + InGroup<DanglingField>; def warn_init_ptr_member_to_parameter_addr : Warning< - "initializing pointer member %0 with the stack address of " - "%select{variable|parameter}2 %1">, InGroup<DanglingField>; + "initializing pointer member %0 with the stack address of parameter %1">, + InGroup<DanglingField>; def note_ref_or_ptr_member_declared_here : Note< "%select{reference|pointer}0 member declared here">; |