Author: ericwf
Date: Wed Dec 23 01:04:32 2015
New Revision: 256323
URL: http://llvm.org/viewvc/llvm-project?rev=256323&view=rev
Log:
Add new tests for throwing incomplete pointer types
Modified:
libcxxabi/trunk/test/incomplete_type.sh.cpp
Modified: libcxxabi/trunk/test/incomplete_type.sh.cpp
Author: ericwf
Date: Wed Dec 23 00:35:25 2015
New Revision: 256322
URL: http://llvm.org/viewvc/llvm-project?rev=256322&view=rev
Log:
Fix PR25898 - Check for incomplete pointers types in can_catch(...)
Added:
libcxxabi/trunk/test/incomplete_type.sh.cpp
Modified:
libcxxabi/trunk/src/private
Author: ctopper
Date: Tue Dec 22 23:44:43 2015
New Revision: 256318
URL: http://llvm.org/viewvc/llvm-project?rev=256318&view=rev
Log:
[Sema] Use available enum types instead of integers. As one is used in a
switch, this makes the compiler ensure the switch is fully covered. NFC
Modified:
cfe
Author: ctopper
Date: Tue Dec 22 23:44:46 2015
New Revision: 256319
URL: http://llvm.org/viewvc/llvm-project?rev=256319&view=rev
Log:
[Sema] Make an enum local to the only method that uses it. NFC
Modified:
cfe/trunk/lib/Sema/SemaChecking.cpp
Modified: cfe/trunk/lib/Sema/SemaChecking.cpp
URL
LegalizeAdulthood created this revision.
LegalizeAdulthood added a reviewer: alexfh.
LegalizeAdulthood added a subscriber: cfe-commits.
This changeset still emits the diagnostic that the expression could be
simplified, but it doesn't generate any fix-its that would lose comments or
preprocessor
faisalv added a subscriber: Nate.
Comment at: include/clang/AST/DeclTemplate.h:836
@@ -835,2 +835,3 @@
+ bool IsConcept : 1;
protected:
rsmith wrote:
> This might make more sense on `TemplateDecl`, since we also want this flag
> for `VarTemplateDecl`s. In any
On Tue, Dec 15, 2015 at 2:56 PM, Nico Weber wrote:
> Looks like the warning is still emitted frequently. It looks like cl.exe
> processes /W4 before /wd4100 even though if it's passed as "/wd4100 /W4".
> clang-cl probably interprets this as "first, disable Wunused-parameter,
> then turn on everyt
thakis added a comment.
I've looked at it some, and couldn't find a way of escaping quotes that lit
doesn't strip but that ubsan accepts.
'"c:\foo"' gets its quotes stripped by lit, so does """c:\foo""". \"c:\foo\"
confuses lit enough that it tries to call lit.util.warning() which doesn't
exis
Hello everyone,
Below are some buildbot numbers for the last week of 12/13/2015 -
12/19/2015.
Thanks
Galina
Number of commits by project:
project | commits
---+---
llvm | 293
cfe |
thakis added a comment.
And this in turn is because env strips the quotes:
C:\src\chrome\src\third_party\llvm-bootstrap>env
UBSAN_OPTIONS=halt_on_error=1:suppressions="C:\src\chrome\src\third_party\llvm-bootstrap\projects\compiler-rt\test\ubsan\Standalone-x86_64\TestCases\Integer\Output\suppress
kfunk added a comment.
By the way; prove that it really works:
http://wstaw.org/m/2015/12/23/kdevelop-clazy.png (loving it!) :)
http://reviews.llvm.org/D15729
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/m
thakis added a comment.
It's because FlagParser::is_space() considers ':' as a space, so it thinks c:\
ends after the c and then starts parsing another flag at the \.
Fix ideas: Don't treat : as space while in quotes, or use ; as separator on
Windows.
Repository:
rL LLVM
http://reviews.llv
kfunk added a comment.
Doing as requested by
http://lists.llvm.org/pipermail/cfe-dev/2014-October/039381.html, let's put
this up for review in its initial form, so we can discuss a better approach.
http://reviews.llvm.org/D15729
___
cfe-commits ma
thakis added a comment.
Here's the error (I still don't know how to run single tests through lit,
though):
C:\src\chrome\src\third_party\llvm-bootstrap>env
UBSAN_OPTIONS=halt_on_error=1:suppressions="C:\src\chrome\src\third_party\llvm-bootstrap\projects\compiler-rt\test\ubsan\Standalone-x86_64\
thakis added a subscriber: thakis.
Comment at: compiler-rt/trunk/test/ubsan/TestCases/Integer/suppressions.cpp:1
@@ +1,2 @@
+// RUN: %clangxx -fsanitize=integer -g0 %s -o %t
+
This test fails on Windows. I tried to debug, but the usual way to run tests
seems to n
Well yes, but we have a Windows bot on http://lab.llvm.org:8011/console,
right?
On Tue, Dec 22, 2015 at 5:54 PM, Dan Gohman wrote:
> It appears the reason it was only caught on Windows was that only Windows
> appends ".exe" to executable files. Your change to match lld with {{.*}}
> appended fix
davidxl added a comment.
This looks like a straightforward cleanup (the test cases have been in a
previous patch).
http://reviews.llvm.org/D15726
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo
Author: nico
Date: Tue Dec 22 16:42:56 2015
New Revision: 256288
URL: http://llvm.org/viewvc/llvm-project?rev=256288&view=rev
Log:
Disable include sorting by default for Chromium style.
Include sorting generally can break .cc files, especially on Windows. Make
this opt-in for Chromium style to g
Author: adrian
Date: Tue Dec 22 16:37:22 2015
New Revision: 256287
URL: http://llvm.org/viewvc/llvm-project?rev=256287&view=rev
Log:
Document that we recommend to turn off -gmodules when building a static
library for distribution to other machines on the clang man page.
Modified:
cfe/trunk/do
This might have broken the test on Windows:
Command 1 Stderr:
C:\src\chrome\src\third_party\llvm\tools\clang\test\Driver\wasm-toolchain.c:3:13:
error: expected string not found in in
put
// AS_LINK: lld" "-flavor" "ld" "[[temp]]" "-o" "a.out"
^
:5:476: note: scanning from here
"C:/src
Author: nico
Date: Tue Dec 22 16:30:14 2015
New Revision: 256286
URL: http://llvm.org/viewvc/llvm-project?rev=256286&view=rev
Log:
Make wasm-toolchain.c test pass on Windows.
Modified:
cfe/trunk/test/Driver/wasm-toolchain.c
Modified: cfe/trunk/test/Driver/wasm-toolchain.c
URL:
http://llvm.o
Sure, will do
On Dec 17, 2015 6:44 PM, "Aaron Ballman" wrote:
> Can you also be sure to update the release notes for this change?
>
> ~Aaron
>
> On Thu, Dec 17, 2015 at 6:49 AM, Alexander Kornienko via cfe-commits
> wrote:
> > Author: alexfh
> > Date: Thu Dec 17 05:49:19 2015
> > New Revision: 2
rsmith added inline comments.
Comment at: include/clang/AST/DeclTemplate.h:836
@@ -835,2 +835,3 @@
+ bool IsConcept : 1;
protected:
This might make more sense on `TemplateDecl`, since we also want this flag for
`VarTemplateDecl`s. In any case, please use some
eraman created this revision.
eraman added reviewers: bogner, davidxl.
eraman added a subscriber: cfe-commits.
NFC. These hints are only used for inlining and the inliner now uses the same
criteria to identify hot and cold callees and set appropriate thresholds
without relying on these hints. He
On Tue, Dec 22, 2015 at 3:16 PM, Aaron Ballman
wrote:
> On Tue, Dec 22, 2015 at 3:06 PM, Samuel Benzaquen via cfe-commits
> wrote:
> > Author: sbenza
> > Date: Tue Dec 22 14:06:40 2015
> > New Revision: 256278
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=256278&view=rev
> > Log:
> > [AST
Author: sbenza
Date: Tue Dec 22 15:06:36 2015
New Revision: 256284
URL: http://llvm.org/viewvc/llvm-project?rev=256284&view=rev
Log:
[ASTMatchers] Fix typo in booleanType() doc.
Fix typo in booleanType() doc and recreate the
LibASTMatchersReference.html reference document.
Modified:
cfe/trun
On Tue, Dec 22, 2015 at 3:06 PM, Samuel Benzaquen via cfe-commits
wrote:
> Author: sbenza
> Date: Tue Dec 22 14:06:40 2015
> New Revision: 256278
>
> URL: http://llvm.org/viewvc/llvm-project?rev=256278&view=rev
> Log:
> [ASTMatchers] Add booleanType() matcher.
>
> Modified:
> cfe/trunk/include
Author: sbenza
Date: Tue Dec 22 14:06:40 2015
New Revision: 256278
URL: http://llvm.org/viewvc/llvm-project?rev=256278&view=rev
Log:
[ASTMatchers] Add booleanType() matcher.
Modified:
cfe/trunk/include/clang/ASTMatchers/ASTMatchers.h
cfe/trunk/lib/ASTMatchers/Dynamic/Registry.cpp
cfe/
nwilson updated this revision to Diff 43469.
nwilson added a comment.
Updating to r256247
http://reviews.llvm.org/D13357
Files:
include/clang/AST/DeclTemplate.h
include/clang/Basic/DiagnosticSemaKinds.td
lib/Sema/SemaDecl.cpp
lib/Sema/SemaTemplate.cpp
test/CXX/concepts-ts/dcl.dcl/dcl.
george.burgess.iv created this revision.
george.burgess.iv added a reviewer: Anastasia.
george.burgess.iv added a subscriber: cfe-commits.
Clang generally treats booleans as 8-bit types, but lowers them to 1-bit types.
This means we currently happily accept C++ code like:
```
typedef __attribute
danielaustin updated this revision to Diff 43459.
danielaustin added a comment.
Test case added which looks for the existence of multiple trap basic blocks in
generated LLVM-IR with optimization turned on. Flag renamed
(-fmerge-traps/-fno-merge-traps) and placed in a more sensible location
(Cod
Author: alexfh
Date: Tue Dec 22 12:13:00 2015
New Revision: 256261
URL: http://llvm.org/viewvc/llvm-project?rev=256261&view=rev
Log:
[clang-tidy] Added documentation for modernize-redundant-void-arg
Added:
clang-tools-extra/trunk/docs/clang-tidy/checks/modernize-redundant-void-arg.rst
Modifi
Author: alexfh
Date: Tue Dec 22 11:36:49 2015
New Revision: 256259
URL: http://llvm.org/viewvc/llvm-project?rev=256259&view=rev
Log:
[clang-tidy] Updates to documentation: formatting, titles, etc.
Modified:
clang-tools-extra/trunk/clang-tidy/add_new_check.py
clang-tools-extra/trunk/docs/c
Author: alexfh
Date: Tue Dec 22 11:10:34 2015
New Revision: 256254
URL: http://llvm.org/viewvc/llvm-project?rev=256254&view=rev
Log:
Updated year (2014-2015).
Modified:
clang-tools-extra/trunk/LICENSE.TXT
clang-tools-extra/trunk/docs/conf.py
Modified: clang-tools-extra/trunk/LICENSE.TXT
Author: alexfh
Date: Tue Dec 22 11:08:57 2015
New Revision: 256253
URL: http://llvm.org/viewvc/llvm-project?rev=256253&view=rev
Log:
[clang-tools-extra] Text formatting
Modified:
clang-tools-extra/trunk/docs/index.rst
Modified: clang-tools-extra/trunk/docs/index.rst
URL:
http://llvm.org/vie
Thanks!
On 21 December 2015 at 18:30, Dan Gohman via cfe-commits
wrote:
> Author: djg
> Date: Mon Dec 21 17:30:41 2015
> New Revision: 256216
>
> URL: http://llvm.org/viewvc/llvm-project?rev=256216&view=rev
> Log:
> [WebAssembly] Remove the -target command-line flag from the ld commandline.
>
> T
Can't you use the preprocessor to produce a large output? See the
testcase in https://llvm.org/bugs/show_bug.cgi?id=10651#c10 for
example.
Cheers,
Rafael
On 21 December 2015 at 20:07, Yunzhong Gao via cfe-commits
wrote:
> ygao created this revision.
> ygao added subscribers: cfe-commits, rsandi
Author: djasper
Date: Tue Dec 22 09:48:35 2015
New Revision: 256247
URL: http://llvm.org/viewvc/llvm-project?rev=256247&view=rev
Log:
clang-format: [JS] Support arrays of object-type literals.
Before:
interface I {
o: {}
[];
}
After:
interface I {
o: {}[];
}
Modified:
cf
Author: djasper
Date: Tue Dec 22 09:48:15 2015
New Revision: 256246
URL: http://llvm.org/viewvc/llvm-project?rev=256246&view=rev
Log:
clang-format: [JS] Conservatively introduce column layout for JS array
initializers. For now, only use it for 20 items or more. Otherwise,
clang-format formats thes
Author: djasper
Date: Tue Dec 22 09:47:56 2015
New Revision: 256245
URL: http://llvm.org/viewvc/llvm-project?rev=256245&view=rev
Log:
clang-format: [JS] "operator" is not a keyword in Java/JavaScript.
Modified:
cfe/trunk/lib/Format/Format.cpp
cfe/trunk/unittests/Format/FormatTestJS.cpp
alexfh added inline comments.
Comment at: clang-tidy/google/DefinitionsInHeadersCheck.cpp:23
@@ +22,3 @@
+bool inHeaderFile(const SourceManager* SM,
+ const SourceLocation& Location) {
+ StringRef Filename = SM->getFilename(SM->getExpansionLoc(Location));
---
tberghammer abandoned this revision.
tberghammer added a comment.
Thank you for adding the new debugger tuning feature. I am using LLDB so
"-glldb" will work for me perfectly.
There is one thing we might consider is that the accelerator tables increasing
the size of the binary by ~10% (measured
Thanks!
Committed with a slight modification as address spaces are now handled
correctly by convertToAtomicIntPointer helper method!
Cheers
Anastasia
-Original Message-
From: Pekka Jääskeläinen [mailto:pekka.jaaskelai...@tut.fi]
Sent: 17 December 2015 08:35
To: Anastasia Stulova; cfe
Author: stulova
Date: Tue Dec 22 09:14:54 2015
New Revision: 256243
URL: http://llvm.org/viewvc/llvm-project?rev=256243&view=rev
Log:
[OpenCL] Fix atomic Builtins check for address spaces of non-atomic pointer
If there are two pointers passed to an atomic Builtin,
Clang doesn't allow the second
logan closed this revision.
logan added a comment.
Thanks! Committed as http://reviews.llvm.org/rL256241.
http://reviews.llvm.org/D15613
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm
Author: logan
Date: Tue Dec 22 08:38:30 2015
New Revision: 256241
URL: http://llvm.org/viewvc/llvm-project?rev=256241&view=rev
Log:
Fix ARM __cxa_end_cleanup() and gc-sections.
This commit adds SHF_ALLOC and SHF_EXECINSTR section flags to
`.text.__cxa_end_cleanup` section. This fixes a link erro
a.makarov updated this revision to Diff 43440.
a.makarov added a comment.
I've updated the patch. Please, re-review it again.
About creating attributed type - I think we shouldn't create it if we ignored
specified CC attribute. Ignoring specified CC attribute (and emitting the
warning, of course
jrmarino added a subscriber: jrmarino.
jrmarino added a comment.
Hi, can someone take a look at http://reviews.llvm.org/D15166?
It's a simple patch but critical for DragonFly.
http://reviews.llvm.org/D15166
___
cfe-commits mailing list
cfe-commits@
jrmarino added a comment.
Hi Davide,
Yes, the patch is tested it. We are carrying them in FreeBSD ports (which
DragonFly uses), so they are being used now.
http://reviews.llvm.org/D15166
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
htt
a.makarov updated this revision to Diff 43442.
a.makarov added a comment.
Actual updated patch. Sorry for noise.
http://reviews.llvm.org/D15373
Files:
include/clang/Sema/Sema.h
lib/Sema/SemaDeclAttr.cpp
lib/Sema/SemaType.cpp
test/CodeGen/adding_defaulted_cc_attr_to_type.c
test/Sema/ca
berenm added a comment.
Actually, the main issue that I see is the difficulty to build the plugin
itself. The project does not build out-of-the-box on VS2013, and I had to
change the assembly references by hand.
I have a patch to get the required assemblies in a more portable way using
nuget p
berenm marked 8 inline comments as done.
berenm added a comment.
For information, I just tested the current diff under Visual Studio 2010 SP1
and everything looks fine as well.
http://reviews.llvm.org/D12407
___
cfe-commits mailing list
cfe-commits
Author: dsanders
Date: Tue Dec 22 06:59:30 2015
New Revision: 256240
URL: http://llvm.org/viewvc/llvm-project?rev=256240&view=rev
Log:
[mips] Add _GCC_HAVE_SYNC_COMPARE_AND_SWAP macros.
This fixes the 'pure virtual function called' failure with ThreadPool in a
clang-built clang. This fixes the ll
Author: abataev
Date: Tue Dec 22 06:44:46 2015
New Revision: 256239
URL: http://llvm.org/viewvc/llvm-project?rev=256239&view=rev
Log:
[OPENMP] Revert r256238 to fix the problem with tests on Linux.
Modified:
cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td
cfe/trunk/include/clang/Basi
Author: abataev
Date: Tue Dec 22 06:21:47 2015
New Revision: 256238
URL: http://llvm.org/viewvc/llvm-project?rev=256238&view=rev
Log:
[OPENMP 4.5] Parsing/sema for 'depend(sink:vec)' clause in 'ordered' directive.
OpenMP 4.5 adds 'depend(sink:vec)' in 'ordered' directive for doacross loop
synchro
davide added a subscriber: davide.
davide accepted this revision.
davide added a reviewer: davide.
davide added a comment.
This revision is now accepted and ready to land.
The patch looks good, and the changes are localized to the DflyBSD driver so if
you tested and works, I'm fine with it. I'll
xazax.hun marked 9 inline comments as done.
xazax.hun added a comment.
It was tested on gcc and rAthena (https://github.com/rathena/rathena). It did
not find any issues in those projects, but I was able to find some issues that
I artificially put into those projects.
Comment a
57 matches
Mail list logo
