Re: [PATCH] D24584: Do not warn about format strings that are indexed string literals.

2016-09-15 Thread Stephen Hines via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL281686: Do not warn about format strings that are indexed string literals. (authored by srhines). Changed prior to commit: https://reviews.llvm.org/D24584?vs=71585&id=71586#toc Repository: rL LLVM h

Re: [PATCH] D24584: Do not warn about format strings that are indexed string literals.

2016-09-15 Thread Meike Baumgärtner via cfe-commits
meikeb updated this revision to Diff 71585. meikeb added a comment. . https://reviews.llvm.org/D24584 Files: lib/Sema/SemaChecking.cpp test/Sema/format-strings.c Index: test/Sema/format-strings.c === --- test/Sema/format-strin

Re: [PATCH] D24584: Do not warn about format strings that are indexed string literals.

2016-09-15 Thread Meike Baumgärtner via cfe-commits
meikeb updated this revision to Diff 71584. meikeb added a comment. Rebase https://reviews.llvm.org/D24584 Files: lib/CodeGen/CGBlocks.cpp lib/CodeGen/CGBlocks.h lib/Sema/SemaChecking.cpp test/CodeGenObjCXX/lambda-expressions.mm test/Sema/format-strings.c Index: test/Sema/format-stri

Re: [PATCH] D24584: Do not warn about format strings that are indexed string literals.

2016-09-15 Thread Meike Baumgärtner via cfe-commits
meikeb updated this revision to Diff 71583. meikeb added a comment. Try to drop randomly uploaded commit. https://reviews.llvm.org/D24584 Files: lib/Sema/SemaChecking.cpp test/Sema/format-strings.c Index: test/Sema/format-strings.c ==

Re: [PATCH] D24584: Do not warn about format strings that are indexed string literals.

2016-09-15 Thread Meike Baumgärtner via cfe-commits
meikeb updated this revision to Diff 71581. meikeb added a comment. Rebase to current commit. https://reviews.llvm.org/D24584 Files: lib/CodeGen/CGBlocks.cpp lib/CodeGen/CGBlocks.h lib/Sema/SemaChecking.cpp test/CodeGenObjCXX/lambda-expressions.mm test/Sema/format-strings.c Index: te

Re: [PATCH] D24584: Do not warn about format strings that are indexed string literals.

2016-09-14 Thread Meike Baumgärtner via cfe-commits
meikeb updated this revision to Diff 71438. meikeb added a comment. Fix typos in commit message. https://reviews.llvm.org/D24584 Files: lib/Sema/SemaChecking.cpp test/Sema/format-strings.c Index: test/Sema/format-strings.c ===

Re: [PATCH] D24584: Do not warn about format strings that are indexed string literals.

2016-09-14 Thread Meike Baumgärtner via cfe-commits
meikeb added a comment. This is the same as https://reviews.llvm.org/D23820 besides that I added myself in the commit message as "Patch by". https://reviews.llvm.org/D23820 was reverted in https://reviews.llvm.org/D24579 because srhines' commit took authorship of this patch. https://reviews.l

[PATCH] D24584: Do not warn about format strings that are indexed string literals.

2016-09-14 Thread Meike Baumgärtner via cfe-commits
meikeb created this revision. meikeb added a reviewer: rsmith. meikeb added subscribers: cfe-commits, srhines. The warning for a format string not being a sting literal and therefore being potentially insecure is overly strict for indecies into sting literals. This fix checks if the index into the