laxmansole added a comment.
Ping
https://reviews.llvm.org/D22834
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: rksimon
Date: Sat Aug 20 15:21:27 2016
New Revision: 279382
URL: http://llvm.org/viewvc/llvm-project?rev=279382&view=rev
Log:
Wdocumentation fix
Modified:
cfe/trunk/lib/Analysis/CloneDetection.cpp
Modified: cfe/trunk/lib/Analysis/CloneDetection.cpp
URL:
http://llvm.org/viewvc/llvm-p
It cycled back to green. flake?
On Sat, Aug 20, 2016 at 7:27 PM, Vitaly Buka wrote:
> msan is not happy about this change
> http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/15681/steps/check-llvm%20msan/logs/stdio
>
> On Sat, Aug 20, 2016 at 9:59 AM Benjamin Kramer via cfe-com
This revision was automatically updated to reflect the committed changes.
Closed by commit rL279378: [analyzer] Use faster hashing (MD5) in
CloneDetector. (authored by dergachev).
Changed prior to commit:
https://reviews.llvm.org/D22515?vs=68778&id=68784#toc
Repository:
rL LLVM
https://revi
Author: dergachev
Date: Sat Aug 20 12:35:53 2016
New Revision: 279378
URL: http://llvm.org/viewvc/llvm-project?rev=279378&view=rev
Log:
[analyzer] Use faster hashing (MD5) in CloneDetector.
This replaces the old approach of fingerprinting every AST node into a string,
which avoided collisions and
alexshap added a comment.
ping
Repository:
rL LLVM
https://reviews.llvm.org/D23279
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
msan is not happy about this change
http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/15681/steps/check-llvm%20msan/logs/stdio
On Sat, Aug 20, 2016 at 9:59 AM Benjamin Kramer via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: d0k
> Date: Sat Aug 20 11:51:33 2016
> N
alexfh requested changes to this revision.
This revision now requires changes to proceed.
Comment at:
docs/clang-tidy/checks/cppcoreguidelines-pro-bounds-constant-array-index.rst:11
@@ -10,3 +10,3 @@
-The check can generate fixes after the option `GslHeader` has been set
+The c
Author: d0k
Date: Sat Aug 20 11:51:33 2016
New Revision: 279374
URL: http://llvm.org/viewvc/llvm-project?rev=279374&view=rev
Log:
[Sema] Don't crash on scanf on forward-declared enums.
This is valid in GNU C, which allows pointers to incomplete enums. GCC
just pretends that the underlying type is
v.g.vassilev added inline comments.
Comment at: include/clang/Analysis/CloneDetection.h:235
@@ +234,3 @@
+ /// respective virtual methods.
+ class Constraint {
+ public:
I am still unsure about the name. It is too generic. What about
`CloneConstraint`?
https
WuZhao added a comment.
I wan to point to some issues, because I am also doing AIX support based on
your patch and I tested and verified.
Firstly, in the lib/Driver/ToolChains.cpp line 3756
case llvm::Triple::ppc64:
addPathIfExists(D, getDriver().SysRoot + getDriver().Dir + "/../lib64",
Author: dergachev
Date: Sat Aug 20 04:57:21 2016
New Revision: 279366
URL: http://llvm.org/viewvc/llvm-project?rev=279366&view=rev
Log:
[analyzer] Make CloneDetector consider template arguments.
For example, code samples `isa(S)' and `isa(S)'
are no longer considered to be clones.
Patch by Rapha
Author: dergachev
Date: Sat Aug 20 05:06:59 2016
New Revision: 279367
URL: http://llvm.org/viewvc/llvm-project?rev=279367&view=rev
Log:
[analyzer] Make CloneDetector consider macro expansions.
So far macro-generated code was treated by the CloneDetector as normal code.
This caused that some macro
mgorny created this revision.
mgorny added reviewers: chapuni, garious.
mgorny added a subscriber: cfe-commits.
Adjust the system lit executable search to look for llvm-lit as that is the
name used in AddLLVM.cmake installed by LLVM.
https://reviews.llvm.org/D23745
Files:
CMakeLists.txt
Ind
mgorny created this revision.
mgorny added reviewers: rsmith, vsk.
mgorny added a subscriber: cfe-commits.
Add a cmake check for sys/resource.h and replace the __has_include() check with
its result, in order to make it possible to use rlimits when building with
compilers not supporting __has_inc
Author: vvassilev
Date: Sat Aug 20 09:50:22 2016
New Revision: 279371
URL: http://llvm.org/viewvc/llvm-project?rev=279371&view=rev
Log:
PR28423: Compare primary declaration contexts.
In certain cases (mostly coming from modules), Sema's idea of the StdNamespace
does not point to the first declara
olesalscheider updated this revision to Diff 68779.
https://reviews.llvm.org/D23746
Files:
lib/Basic/Targets.cpp
Index: lib/Basic/Targets.cpp
===
--- lib/Basic/Targets.cpp
+++ lib/Basic/Targets.cpp
@@ -1955,7 +1955,8 @@
GK_CA
teemperor marked an inline comment as done.
teemperor added a comment.
https://reviews.llvm.org/D22515
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
teemperor updated this revision to Diff 68778.
teemperor added a comment.
- Replaced hash_stream with MD5 in the sub-sequence code.
https://reviews.llvm.org/D22515
Files:
include/clang/Analysis/CloneDetection.h
lib/Analysis/CloneDetection.cpp
lib/StaticAnalyzer/Checkers/CloneChecker.cpp
On Sat, Aug 20, 2016 at 12:45:59AM +, Chandler Carruth wrote:
> I feel like this thread has two very unrelated concerns and it might be
> useful to separate them.
>
> 1) Should an application be changing the stack limit, or should the system
> be able to enforce this?
>
> Fortunately, there a
olesalscheider created this revision.
olesalscheider added reviewers: cfe-commits, tstellarAMD, arsenm.
https://reviews.llvm.org/D23746
Files:
lib/Basic/Targets.cpp
Index: lib/Basic/Targets.cpp
===
--- lib/Basic/Targets.cpp
+++ li
NoQ added inline comments.
Comment at: lib/Analysis/CloneDetection.cpp:550
@@ -548,2 +549,3 @@
CloneDetector::CloneSignature SubSignature;
+llvm::hash_stream SubHash;
Hmm, this still uses `hash_stream`.
https://reviews.llvm.org/D22515
_
This revision was automatically updated to reflect the committed changes.
Closed by commit rL279367: [analyzer] Make CloneDetector consider macro
expansions. (authored by dergachev).
Changed prior to commit:
https://reviews.llvm.org/D23316?vs=68684&id=68771#toc
Repository:
rL LLVM
https://r
This revision was automatically updated to reflect the committed changes.
Closed by commit rL279366: [analyzer] Make CloneDetector consider template
arguments. (authored by dergachev).
Changed prior to commit:
https://reviews.llvm.org/D23555?vs=68683&id=68770#toc
Repository:
rL LLVM
https:/
24 matches
Mail list logo
