hintonda added a comment.
Looks like Manuel cleaned it up along with another change on the 8th, r247018.
thanks again...
http://reviews.llvm.org/D12549
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/li
gribozavr closed this revision.
gribozavr added a comment.
Current trunk does not contain this line, no need to apply the patch.
http://reviews.llvm.org/D12549
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mai
hintonda added a comment.
I don't have commit access, so could you commit it for me?
thanks...
don
http://reviews.llvm.org/D12549
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: xazax
Date: Sun Sep 13 18:03:11 2015
New Revision: 247533
URL: http://llvm.org/viewvc/llvm-project?rev=247533&view=rev
Log:
[Static Analyzer] Remove a redundant file.
Removed:
cfe/trunk/lib/StaticAnalyzer/Checkers/ObjCGenericsChecker.cpp
Removed: cfe/trunk/lib/StaticAnalyzer/Checkers
This revision was automatically updated to reflect the committed changes.
Closed by commit rL247532: [Static Analyzer] Merge the Objective-C Generics
Checker into Dynamic Type… (authored by xazax).
Changed prior to commit:
http://reviews.llvm.org/D12381?vs=34606&id=34651#toc
Repository:
rL L
Author: xazax
Date: Sun Sep 13 18:02:24 2015
New Revision: 247532
URL: http://llvm.org/viewvc/llvm-project?rev=247532&view=rev
Log:
[Static Analyzer] Merge the Objective-C Generics Checker into Dynamic Type
Propagation checker.
Differential Revision: http://reviews.llvm.org/D12381
Modified:
xazax.hun marked 3 inline comments as done.
Comment at: lib/StaticAnalyzer/Checkers/DynamicTypePropagation.cpp:121
@@ -54,3 +120,3 @@
I != E; ++I) {
if (!SR.isLiveRegion(I->first)) {
State = State->remove(I->first);
zaks.anna wrote:
> It's odd
gribozavr accepted this revision.
gribozavr added a comment.
This revision is now accepted and ready to land.
LGTM.
http://reviews.llvm.org/D12549
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo
Author: vedantk
Date: Sun Sep 13 15:17:18 2015
New Revision: 247530
URL: http://llvm.org/viewvc/llvm-project?rev=247530&view=rev
Log:
[docs] NFC: Fix line formatting in INSTALL.txt
Patch by beltex!
Differential Revision: http://reviews.llvm.org/D12684
Modified:
cfe/trunk/INSTALL.txt
Modifi
This revision was automatically updated to reflect the committed changes.
Closed by commit rL247530: [docs] NFC: Fix line formatting in INSTALL.txt
(authored by vedantk).
Changed prior to commit:
http://reviews.llvm.org/D12684?vs=34178&id=34650#toc
Repository:
rL LLVM
http://reviews.llvm.or
Alexander_Droste updated this revision to Diff 34647.
Alexander_Droste marked 2 inline comments as done.
Alexander_Droste added a comment.
- fixed integration test file: tools/clang/test/Analysis/MPIChecker.c
- mocked types, constants, functions from mpi.h
- mocked types from stdint.h
- dropped -I
beltex added a comment.
Thanks! My first patch! :)
http://reviews.llvm.org/D12684
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
RKSimon created this revision.
RKSimon added reviewers: ab, qcolombet, craig.topper, spatel.
RKSimon added a subscriber: cfe-commits.
RKSimon set the repository for this revision to rL LLVM.
128-bit vector integer sign extensions correctly lower to the pmovsx
instructions even for debug builds.
I've also found a bunch of similar cases in our codebase, and I'm trying to
figure out whether the pattern can be narrowed down to just dangerous
cases. If we don't find a way to do so, we'll probably have to resort to
"// NOLINT" to shut clang-tidy up.
On 13 Sep 2015 10:52, "Kim Gräsman" wrote:
Late to the party, but I wanted to ask: is there a way to indicate to
the checker that we really *did* mean sizeof()?
I think I've stumbled over code in our code base that uses
sizeof(container) to report memory usage statistics and it seems
valid, so it'd be nice if this checker could be silenced
ikudrin created this revision.
ikudrin added reviewers: ruiu, rnk.
ikudrin added a subscriber: cfe-commits.
With Windows 10 SDK, Include and Lib directories now contain an additional
subfolder with the name that corresponds to the full version of the SDK, for
example:
- C:\Program Files (x86)\
16 matches
Mail list logo