On Mon, Sep 07, 2015 at 02:44:23PM -0700, Saleem Abdulrasool wrote:
> > Right. Tried to explain a bit what's going on in the commit message,
> > with a reference to the original commit that removed the flag.
>
>
> Thanks. Would you like me to commit this on your behalf as well?
Yes please. Than
comex created this revision.
comex added a subscriber: cfe-commits.
Herald added subscribers: aemerson, klimek.
Add support for GCC's '__auto_type' extension.
As per the GCC manual: https://gcc.gnu.org/onlinedocs/gcc/Typeof.html
Implemented in GCC 4.9, __auto_type is similar to C++11 auto but wo
seaneveson updated this revision to Diff 34196.
seaneveson added a comment.
I changed the name to min-cfg-size-treat-functions-as-large, thanks for the
suggestion.
If the patch is acceptable can someone commit it for me?
http://reviews.llvm.org/D12406
Files:
include/clang/StaticAnalyzer/Cor
Author: rjmccall
Date: Tue Sep 8 03:21:11 2015
New Revision: 246986
URL: http://llvm.org/viewvc/llvm-project?rev=246986&view=rev
Log:
Move BlockByrefHelpers back to CodeGenModule.h to placate MSVC.
Modified:
cfe/trunk/lib/CodeGen/CGBlocks.h
cfe/trunk/lib/CodeGen/CodeGenModule.h
Modified
Author: rjmccall
Date: Tue Sep 8 03:57:00 2015
New Revision: 246988
URL: http://llvm.org/viewvc/llvm-project?rev=246988&view=rev
Log:
Remove unnecessary braces; this resolves against a
single-pointer overload instead of the ArrayRef one.
Modified:
cfe/trunk/lib/CodeGen/CGBuilder.h
Modified:
klimek accepted this revision.
klimek added a comment.
LG
http://reviews.llvm.org/D12675
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
This revision was automatically updated to reflect the committed changes.
Closed by commit rL246989: Avoid using rvalue references with trivially
copyable types. (authored by angelgarcia).
Changed prior to commit:
http://reviews.llvm.org/D12675?vs=34162&id=34201#toc
Repository:
rL LLVM
http
Author: angelgarcia
Date: Tue Sep 8 04:01:21 2015
New Revision: 246989
URL: http://llvm.org/viewvc/llvm-project?rev=246989&view=rev
Log:
Avoid using rvalue references with trivially copyable types.
Summary:
When the dereference operator returns a value that is trivially
copyable (like a pointer)
Author: rjmccall
Date: Tue Sep 8 04:18:30 2015
New Revision: 246991
URL: http://llvm.org/viewvc/llvm-project?rev=246991&view=rev
Log:
When building the alloca for a local variable, set its name
separately from building the instruction so that it's
preserved even in -Asserts builds.
Employ C++'s
Author: chapuni
Date: Tue Sep 8 04:31:04 2015
New Revision: 246992
URL: http://llvm.org/viewvc/llvm-project?rev=246992&view=rev
Log:
Fix clang/test/CodeGenCXX/alignment.cpp for -Asserts.
Modified:
cfe/trunk/test/CodeGenCXX/alignment.cpp
Modified: cfe/trunk/test/CodeGenCXX/alignment.cpp
URL:
Author: rjmccall
Date: Tue Sep 8 04:33:33 2015
New Revision: 246993
URL: http://llvm.org/viewvc/llvm-project?rev=246993&view=rev
Log:
Another fix to this test, this one apparently working by
coincidence on all bots.
Modified:
cfe/trunk/test/CodeGenCXX/alignment.cpp
Modified: cfe/trunk/test/
Author: chapuni
Date: Tue Sep 8 04:37:09 2015
New Revision: 246994
URL: http://llvm.org/viewvc/llvm-project?rev=246994&view=rev
Log:
Fix clang/test/CodeGen/mips-varargs.c for -Asserts, possibly typo.
Modified:
cfe/trunk/test/CodeGen/mips-varargs.c
Modified: cfe/trunk/test/CodeGen/mips-varar
Yea, we had that discussion a few times, and I can never remember why we
ended up in the state we're in.
We definitely had a time where we switched to just using the exact same
name as the node's class name for the matchers.
I *think* we didn't do it for cxxRecordDecl, because Richard said that's a
Author: chapuni
Date: Tue Sep 8 04:42:41 2015
New Revision: 246995
URL: http://llvm.org/viewvc/llvm-project?rev=246995&view=rev
Log:
clangCodeGen: Fix comments. [-Wdocumentation]
Modified:
cfe/trunk/lib/CodeGen/CGClass.cpp
cfe/trunk/lib/CodeGen/CGDecl.cpp
cfe/trunk/lib/CodeGen/CGOpen
Author: alexfh
Date: Tue Sep 8 04:44:04 2015
New Revision: 246996
URL: http://llvm.org/viewvc/llvm-project?rev=246996&view=rev
Log:
[clang-tidy] Updated docs.
Added:
clang-tools-extra/trunk/docs/clang-tidy/checks/modernize-replace-auto-ptr.rst
clang-tools-extra/trunk/docs/clang-tidy/che
eastig created this revision.
eastig added a subscriber: cfe-commits.
When an executable (e.g. for bare metal environment) is statically linked with
libc++ the following code might not work as expected:
```
#include
class Foo {
public:
Foo(int n) {
std::cout << "Hello World - " << n <<
Author: klimek
Date: Tue Sep 8 05:11:26 2015
New Revision: 246998
URL: http://llvm.org/viewvc/llvm-project?rev=246998&view=rev
Log:
Fix documentation of numSelectorArgs.
Currently, the documentation for numSelectorArgs includes an incorrect
example. It shows a case where an argument of 1 will ma
klimek accepted this revision.
klimek added a reviewer: klimek.
klimek added a comment.
This revision is now accepted and ready to land.
LG, as this is a documentation change that looks about right, and comes with
tests, and the original author doesn't jump in.
http://reviews.llvm.org/D12471
klimek closed this revision.
klimek added a comment.
Committed as r246998.
http://reviews.llvm.org/D12471
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: klimek
Date: Tue Sep 8 05:31:09 2015
New Revision: 247001
URL: http://llvm.org/viewvc/llvm-project?rev=247001&view=rev
Log:
Update code owners for AST matchers / libtooling.
Modified:
cfe/trunk/CODE_OWNERS.TXT
Modified: cfe/trunk/CODE_OWNERS.TXT
URL:
http://llvm.org/viewvc/llvm-pro
Author: alexfh
Date: Tue Sep 8 05:31:36 2015
New Revision: 247002
URL: http://llvm.org/viewvc/llvm-project?rev=247002&view=rev
Log:
[clang-tidy] Fix run-clang-tidy.py.
Do not add "-*" to the list of checks. Make consistent the list of enabled
checks and the checks in use. Moreover, removing "-*"
kuhar updated this revision to Diff 34206.
kuhar added a comment.
Merge with the latest changes to CGCall.cpp
Repository:
rL LLVM
http://reviews.llvm.org/D12400
Files:
lib/CodeGen/CGCall.cpp
test/CodeGen/arm_function_epilog.cpp
Index: test/CodeGen/arm_function_epilog.cpp
===
Author: kuhar
Date: Tue Sep 8 05:36:42 2015
New Revision: 247003
URL: http://llvm.org/viewvc/llvm-project?rev=247003&view=rev
Log:
findDominatingStoreToReturn in CGCall.cpp didn't check if a candidate store
instruction used the ReturnValue as pointer operand or value operand. This
led to wrong co
This revision was automatically updated to reflect the committed changes.
Closed by commit rL247003: findDominatingStoreToReturn in CGCall.cpp didn't
check if a candidate store (authored by kuhar).
Changed prior to commit:
http://reviews.llvm.org/D12400?vs=34206&id=34207#toc
Repository:
rL L
matto1990 accepted this revision.
matto1990 added a comment.
This revision is now accepted and ready to land.
Looks good to me now!
http://reviews.llvm.org/D12369
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/
On Fri, Aug 28, 2015 at 07:52:17PM +0200, Xan López via cfe-commits wrote:
> There is no __cxa_finalize symbol available on recent Solaris OS
> versions, so we need this flag to make non trivial C++ programs run.
What do you consider as recent Solaris? When I asked, I've been told
that Illumos has
On Mon, Sep 07, 2015 at 11:50:55PM +, Eric Fiselier via cfe-commits wrote:
> EricWF added a comment.
>
> Why would we not use "-isysroot" on other platforms as well?
Because otherwise you can't link in a cross-compiling setup?
Joerg
___
cfe-commits
kubabrecka added a comment.
In what scenario exactly are you seeing an issue? If it's a just-built clang
that can't find C++ headers, then you should just build the libcxx project
alongside.
Comment at: lib/Driver/ToolChains.cpp:290-295
@@ +289,8 @@
+
+ // FIXME: We shou
On Tue, Sep 08, 2015 at 01:09:02PM +0200, Joerg Sonnenberger via cfe-commits
wrote:
> On Fri, Aug 28, 2015 at 07:52:17PM +0200, Xan López via cfe-commits wrote:
> > There is no __cxa_finalize symbol available on recent Solaris OS
> > versions, so we need this flag to make non trivial C++ programs
On Tue, Sep 08, 2015 at 01:27:42PM +0200, Xan López via cfe-commits wrote:
> > What do you consider as recent Solaris? When I asked, I've been told
> > that Illumos has been providing __cxa_atexit since 2013. As such,
> > disabling it for the Solaris family by default seems just wrong.
>
> Solaris
On Tue, Sep 8, 2015 at 5:40 AM, Manuel Klimek wrote:
> Yea, we had that discussion a few times, and I can never remember why we
> ended up in the state we're in.
> We definitely had a time where we switched to just using the exact same name
> as the node's class name for the matchers.
> I *think*
Author: alexfh
Date: Tue Sep 8 07:26:32 2015
New Revision: 247007
URL: http://llvm.org/viewvc/llvm-project?rev=247007&view=rev
Log:
Add a redirection page to unbreak external links.
Apparently, there are some links to http://clang.llvm.org/extra/clang-tidy.html
in the wild. It's better to keep t
hintonda added a comment.
In http://reviews.llvm.org/D12646#241308, @kubabrecka wrote:
> In what scenario exactly are you seeing an issue? If it's a just-built clang
> that can't find C++ headers, then you should just build the libcxx project
> alongside.
clang++ defaults to -stdlib=libc++ f
On Tue, Sep 8, 2015 at 2:23 PM Aaron Ballman wrote:
> On Tue, Sep 8, 2015 at 5:40 AM, Manuel Klimek wrote:
> > Yea, we had that discussion a few times, and I can never remember why we
> > ended up in the state we're in.
> > We definitely had a time where we switched to just using the exact same
On Tue, Sep 8, 2015 at 9:18 AM, Manuel Klimek wrote:
> On Tue, Sep 8, 2015 at 2:23 PM Aaron Ballman wrote:
>>
>> On Tue, Sep 8, 2015 at 5:40 AM, Manuel Klimek wrote:
>> > Yea, we had that discussion a few times, and I can never remember why we
>> > ended up in the state we're in.
>> > We definit
On Tue, Sep 8, 2015 at 3:23 PM Aaron Ballman wrote:
> On Tue, Sep 8, 2015 at 9:18 AM, Manuel Klimek wrote:
> > On Tue, Sep 8, 2015 at 2:23 PM Aaron Ballman
> wrote:
> >>
> >> On Tue, Sep 8, 2015 at 5:40 AM, Manuel Klimek
> wrote:
> >> > Yea, we had that discussion a few times, and I can never
On Tue, Sep 08, 2015 at 01:32:21PM +0200, Xan López via cfe-commits wrote:
> On Tue, Sep 08, 2015 at 01:27:42PM +0200, Xan López via cfe-commits wrote:
> > > What do you consider as recent Solaris? When I asked, I've been told
> > > that Illumos has been providing __cxa_atexit since 2013. As such,
On Tue, Sep 08, 2015 at 03:28:12PM +0200, Joerg Sonnenberger via cfe-commits
wro> We don't distinguish Solaris and OpenSolaris forks currently. As such,
> changing one for the other is not a good choice, especially since you
> are more likely to find clang users for Illumos, IMO.
>
> https://www.
aaron.ballman added a comment.
This is looking great! Some mostly minor feedback below. The only things I
noticed that may require further consideration are throwing function parameters
and throwing exception handler variables -- I don't think those scenarios
should emit a diagnostic. The forme
Author: chapuni
Date: Tue Sep 8 09:19:13 2015
New Revision: 247015
URL: http://llvm.org/viewvc/llvm-project?rev=247015&view=rev
Log:
clang/test/CodeGen/complex-convert.c: Appease for targeting powerpc64-*.
Modified:
cfe/trunk/test/CodeGen/complex-convert.c
Modified: cfe/trunk/test/CodeGen/c
ikudrin created this revision.
ikudrin added a subscriber: cfe-commits.
With Visual Studio 2015 release, a part of runtime library was extracted and
now comes with Windows Kits. This patch enables clang to use Universal CRT
library if %INCLUDE or %LIB environment varaibles are not specified.
S
nmusgrave updated this revision to Diff 34220.
nmusgrave marked an inline comment as done.
nmusgrave added a comment.
- Rename test.
http://reviews.llvm.org/D12616
Files:
test/CodeGenCXX/sanitize-dtor-trivial.cpp
Index: test/CodeGenCXX/sanitize-dtor-trivial.cpp
==
Author: klimek
Date: Tue Sep 8 10:14:06 2015
New Revision: 247018
URL: http://llvm.org/viewvc/llvm-project?rev=247018&view=rev
Log:
Fix performance regression when running clang tools.
Brings tool start time for a large synthetic test case down from (on my
machine) 4 seconds to 0.5 seconds.
Mod
In r247018 I get a ~8x speedup on a synthetic benchmark I tried. Can you
validate this fixes the regression?
On Sat, Sep 5, 2015 at 12:56 AM Hans Wennborg wrote:
> On Fri, Aug 14, 2015 at 2:55 AM, Manuel Klimek via cfe-commits
> wrote:
> > Author: klimek
> > Date: Fri Aug 14 04:55:36 2015
> > N
On Tue, Sep 8, 2015 at 2:18 AM, John McCall via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: rjmccall
> Date: Tue Sep 8 04:18:30 2015
> New Revision: 246991
>
> URL: http://llvm.org/viewvc/llvm-project?rev=246991&view=rev
> Log:
> When building the alloca for a local variable, set i
Yes, the run-time is now back to where it was before the regression. Thanks!
On Tue, Sep 8, 2015 at 8:16 AM, Manuel Klimek wrote:
> In r247018 I get a ~8x speedup on a synthetic benchmark I tried. Can you
> validate this fixes the regression?
>
> On Sat, Sep 5, 2015 at 12:56 AM Hans Wennborg wro
kubabrecka added a comment.
> > I first noticed it when I tried to run the version of clang-tidy I'd just
> > built and found I had to pass the path for it to find iostream.
>
>
> I think it works if you a) check out libcxx into llvm/projects/libcxx b) run
> `make install` and c) run the bi
hintonda added a comment.
> I think it works if you a) check out libcxx into llvm/projects/libcxx b) run
> `make install` and c) run the binary from your install directory.
Yes, that works. But why shouldn't users be able to use the installed version?
Also, if you want to force users to eithe
thakis added a comment.
In http://reviews.llvm.org/D12646#241318, @hintonda wrote:
> In http://reviews.llvm.org/D12646#241308, @kubabrecka wrote:
>
> > In what scenario exactly are you seeing an issue? If it's a just-built
> > clang that can't find C++ headers, then you should just build the li
bob.wilson added a comment.
On Darwin platforms, the libc++ headers are expected to be installed alongside
clang. If you're not doing that, then you're building it wrong. Adding more
fallback options for finding the headers just makes things worse, because
instead of a clear failure, you're mor
hintonda added a comment.
In http://reviews.llvm.org/D12646#241493, @bob.wilson wrote:
> On Darwin platforms, the libc++ headers are expected to be installed
> alongside clang. If you're not doing that, then you're building it wrong.
> Adding more fallback options for finding the headers just m
Author: nmusgrave
Date: Tue Sep 8 11:38:17 2015
New Revision: 247025
URL: http://llvm.org/viewvc/llvm-project?rev=247025&view=rev
Log:
Failing test highlighting no poisoning if dtor undeclared.
Summary:
If class or struct has not declared a destructor,
no destructor is emitted, and members are n
> On Sep 8, 2015, at 8:25 AM, David Blaikie wrote:
> On Tue, Sep 8, 2015 at 2:18 AM, John McCall via cfe-commits
> mailto:cfe-commits@lists.llvm.org>> wrote:
> Author: rjmccall
> Date: Tue Sep 8 04:18:30 2015
> New Revision: 246991
>
> URL: http://llvm.org/viewvc/llvm-project?rev=246991&view=re
tra updated this revision to Diff 34230.
tra marked 2 inline comments as done.
tra added a comment.
Implementes Richard Smiths' suggestions:
Fixed attribute order in name mangling.
Replaced std::function with template argument.
http://reviews.llvm.org/D12453
Files:
include/clang/Basic/LangOpt
tra marked 2 inline comments as done.
tra added a comment.
http://reviews.llvm.org/D12453
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.
LGTM i guess...
I'm a tiny bit surprised that you have to handle this here rather than
something else handling this for you... But not a lit surprised...
http://reviews.llvm.org/D12622
tra abandoned this revision.
tra added a comment.
Emitting IR is not sufficient to ensure that the kernels survive GDCE, so the
patch does not work with optimizations on.
http://reviews.llvm.org/D11666 would have to do for now.
http://reviews.llvm.org/D11993
_
On Tue, Sep 8, 2015 at 10:26 AM, John McCall wrote:
> On Sep 8, 2015, at 8:25 AM, David Blaikie wrote:
> On Tue, Sep 8, 2015 at 2:18 AM, John McCall via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
>
>> Author: rjmccall
>> Date: Tue Sep 8 04:18:30 2015
>> New Revision: 246991
>>
>> URL: h
None of my performance concerns were relevant to this change FWIW.
I think the reason that this got "fixed" was because people had a tendancy
to *rely* on the name downstream when we made it always present. =/
Personally, I like having *no* names in a no-asserts build because it
ensures that absol
Author: marshall
Date: Tue Sep 8 12:59:09 2015
New Revision: 247036
URL: http://llvm.org/viewvc/llvm-project?rev=247036&view=rev
Log:
Implementation of Boyer-Moore and Boyer-Moore-Horspool searchers for the LFTS.
Added:
libcxx/trunk/test/std/experimental/func/func.searchers/func.searchers.b
mclow.lists closed this revision.
mclow.lists added a comment.
Landed as revision 247036
http://reviews.llvm.org/D11380
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
echristo added a comment.
Inline comment.
Comment at: test/CodeGenCUDA/link-device-bitcode.cu:23-28
@@ +22,8 @@
+//
+// NVVMReflect is a target-specific pass runs after -emit-llvm prints
+// IR, so we need to check NVPTX to make sure that the pass did happen
+// and __nvvm_refle
On Tue, Sep 8, 2015 at 10:52 AM, Chandler Carruth
wrote:
> None of my performance concerns were relevant to this change FWIW.
>
Good to know - if we wanted to go down this path I figure we could just
provide overloads - StringRef and Twine, the StringRef one could always use
the name, even in -A
I agree with Chandler the default should be no names for -Asserts builds.
What I wanted originally was that the stripping of names be a runtime
choice (since the performance overhead of it was minimal when I measured
it), so that when we need to debug with a production compiler, it would
still be
Hi Sean,
Ted provided more details off-list. He suspects that the problem is that we
likely don't add MemSpaceRegions to the worklist because every region is a
subregion of a MemSpaceRegion, and thus we would invalidate, by default, all
regions that were in the same MemSpace as the regions we w
Hello everyone,
LLVM buildmaster will be restarted after 6 PM Pacific time today.
Thanks
Galina
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mzolotukhin added a comment.
Hi Richard, Hal,
Does the patch look good now?
Thanks,
Michael
http://reviews.llvm.org/D12313
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: adrian
Date: Tue Sep 8 14:20:27 2015
New Revision: 247049
URL: http://llvm.org/viewvc/llvm-project?rev=247049&view=rev
Log:
Module Debugging: Emit debug type information into clang modules.
When -fmodule-format is set to "obj", emit debug info for all types
declared in a module or refere
xazax.hun marked 2 inline comments as done.
xazax.hun added a comment.
There are several fallouts from this review, so I decided to split this patch
up the following way:
1. I created a patch to incorporate the result of this review into
ObjCGenericsChecker: http://reviews.llvm.org/D12701
2. I
> On Sep 8, 2015, at 10:52 AM, Chandler Carruth wrote:
> None of my performance concerns were relevant to this change FWIW.
>
> I think the reason that this got "fixed" was because people had a tendancy to
> *rely* on the name downstream when we made it always present. =/ Personally,
> I like h
Author: dougk
Date: Tue Sep 8 14:29:55 2015
New Revision: 247052
URL: http://llvm.org/viewvc/llvm-project?rev=247052&view=rev
Log:
[Shave]: add a -MT option to moviCompile if there wasn't one
Differential Revision: http://reviews.llvm.org/D12622
Modified:
cfe/trunk/lib/Driver/Tools.cpp
This revision was automatically updated to reflect the committed changes.
Closed by commit rL247052: [Shave]: add a -MT option to moviCompile if there
wasn't one (authored by dougk).
Changed prior to commit:
http://reviews.llvm.org/D12622?vs=34006&id=34239#toc
Repository:
rL LLVM
http://rev
Author: rsmith
Date: Tue Sep 8 14:40:14 2015
New Revision: 247055
URL: http://llvm.org/viewvc/llvm-project?rev=247055&view=rev
Log:
[modules] Write the options records to a separate subblock rather than writing
them directly to the control block. These are fairly large, and in a build with
lots o
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
This seems fine to me.
http://reviews.llvm.org/D12038
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailm
hfinkel added a comment.
In http://reviews.llvm.org/D12313#241697, @mzolotukhin wrote:
> Hi Richard, Hal,
>
> Does the patch look good now?
Looks good to me, but please wait for Richard on the changes he requested.
> Thanks,
> Michael
http://reviews.llvm.org/D12313
_
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
Looks fine to me.
Comment at: lib/CodeGen/CGBuiltin.cpp:114-128
@@ -113,1 +113,17 @@
+static Value *MakeNontemporalStore(CodeGenFunction &CGF, const CallExpr *E) {
+ Value
zaks.anna accepted this revision.
This revision is now accepted and ready to land.
Comment at: lib/StaticAnalyzer/Checkers/ObjCGenericsChecker.cpp:226
@@ +225,3 @@
+/// Inputs:
+/// There is a static lower bound (SL)
+/// There is a static upper bound (SL <: SU)
--
Author: adrian
Date: Tue Sep 8 15:41:52 2015
New Revision: 247068
URL: http://llvm.org/viewvc/llvm-project?rev=247068&view=rev
Log:
Module Debugging: Emit debug type information into clang ObjC modules.
When -fmodule-format is set to "obj", emit debug info for all types
declared in a module or r
tra updated the summary for this revision.
tra updated this revision to Diff 34251.
tra added a comment.
Assume (and test) that NVPTX back-end includes NVVMReflect by default.
http://reviews.llvm.org/D11664
Files:
include/clang/Basic/LangOptions.def
include/clang/Driver/CC1Options.td
lib/
echristo added a comment.
Works for me, thanks.
http://reviews.llvm.org/D11664
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
xazax.hun marked 2 inline comments as done.
xazax.hun added a comment.
http://reviews.llvm.org/D12701
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: rjmccall
Date: Tue Sep 8 16:15:22 2015
New Revision: 247075
URL: http://llvm.org/viewvc/llvm-project?rev=247075&view=rev
Log:
Collect SEH captures in a set instead of a vector to avoid
doing redundant work if a variable is used multiple times.
Fixes PR24751.
Modified:
cfe/trunk/lib/
zatrazz created this revision.
zatrazz added reviewers: t.p.northover, aemerson, pcc, rengolin.
zatrazz added a subscriber: cfe-commits.
Herald added subscribers: rengolin, aemerson.
This patch enables MSan for aarch64/linux.
http://reviews.llvm.org/D12708
Files:
lib/Driver/ToolChains.cpp
Ind
dcoughlin added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/CStringChecker.cpp:843
@@ +842,3 @@
+ if (!Length)
+return true;
+
There doesn't seem to be a test that cares about this returning true (as
compared to false).
Comment
probinson added inline comments.
Comment at: lib/CodeGen/CGDebugInfo.cpp:3263-3264
@@ +3262,4 @@
+ const NamespaceDecl *NSDecl = UD.getNominatedNamespace();
+ if (!NSDecl->isAnonymousNamespace() ||
+ CGM.getTarget().getTriple().isPS4CPU()) {
+DBuilder.createImportedMo
Author: adrian
Date: Tue Sep 8 17:06:40 2015
New Revision: 247081
URL: http://llvm.org/viewvc/llvm-project?rev=247081&view=rev
Log:
Remove unneeded #include.
Modified:
cfe/trunk/lib/CodeGen/CGDebugInfo.cpp
Modified: cfe/trunk/lib/CodeGen/CGDebugInfo.cpp
URL:
http://llvm.org/viewvc/llvm-pro
Author: chapuni
Date: Tue Sep 8 17:47:32 2015
New Revision: 247093
URL: http://llvm.org/viewvc/llvm-project?rev=247093&view=rev
Log:
clang/test/Modules/ModuleDebugInfo.cpp: Add -triple %itanium to appease
ms-targeted builds.
I think DebugInfo tests may avoid MS stuff for now.
Modified:
cfe
rsmith added inline comments.
Comment at: lib/CodeGen/CGDebugInfo.cpp:3263-3264
@@ +3262,4 @@
+ const NamespaceDecl *NSDecl = UD.getNominatedNamespace();
+ if (!NSDecl->isAnonymousNamespace() ||
+ CGM.getTarget().getTriple().isPS4CPU()) {
+DBuilder.createImportedModul
Author: pcc
Date: Tue Sep 8 18:01:30 2015
New Revision: 247098
URL: http://llvm.org/viewvc/llvm-project?rev=247098&view=rev
Log:
CodeGen: Introduce CodeGenModule::CreateMetadataIdentifierForType.
This function can be used to create a metadata identifier for a specific
type. No functionality chan
This revision was automatically updated to reflect the committed changes.
Closed by commit rL247098: CodeGen: Introduce
CodeGenModule::CreateMetadataIdentifierForType. (authored by pcc).
Changed prior to commit:
http://reviews.llvm.org/D12038?vs=32173&id=34276#toc
Repository:
rL LLVM
http:/
nmusgrave created this revision.
nmusgrave added reviewers: eugenis, kcc.
nmusgrave added a subscriber: cfe-commits.
After destruction, invocation of virtual functions prevented
by poisoning vtable pointer.
http://reviews.llvm.org/D12712
Files:
lib/CodeGen/CGClass.cpp
test/CodeGenCXX/sanitiz
ahatanak added a comment.
In http://reviews.llvm.org/D11815#238209, @vkalintiris wrote:
> In http://reviews.llvm.org/D11815#237541, @ahatanak wrote:
>
> > In http://reviews.llvm.org/D11815#236368, @vkalintiris wrote:
> >
> > > In http://reviews.llvm.org/D11815#235394, @ahatanak wrote:
> > >
> > >
Author: dcoughlin
Date: Tue Sep 8 18:50:22 2015
New Revision: 247103
URL: http://llvm.org/viewvc/llvm-project?rev=247103&view=rev
Log:
Revert "[Static Analyzer] BugReporter.cpp:2869: Assertion failed:
!RemainingNodes.empty() && "No error node found in the trimmed graph""
This is making our inte
Author: mzolotukhin
Date: Tue Sep 8 18:52:33 2015
New Revision: 247104
URL: http://llvm.org/viewvc/llvm-project?rev=247104&view=rev
Log:
Introduce __builtin_nontemporal_store and __builtin_nontemporal_load.
Summary:
Currently clang provides no general way to generate nontemporal loads/stores.
Th
This revision was automatically updated to reflect the committed changes.
Closed by commit rL247104: Introduce __builtin_nontemporal_store and
__builtin_nontemporal_load. (authored by mzolotukhin).
Changed prior to commit:
http://reviews.llvm.org/D12313?vs=33492&id=34281#toc
Repository:
rL L
eugenis added inline comments.
Comment at: lib/CodeGen/CGClass.cpp:1671
@@ -1670,1 +1670,3 @@
+ ASTContext &Context = CGF.getContext();
+ // Poison vtable and vtable ptr if they exist for this class.
You are poisoning the vtable pointer in the base des
Author: pcc
Date: Tue Sep 8 19:01:31 2015
New Revision: 247105
URL: http://llvm.org/viewvc/llvm-project?rev=247105&view=rev
Log:
CodeGen: Add CFI unrelated cast checks to the new pointer code path.
Modified:
cfe/trunk/lib/CodeGen/CGExpr.cpp
cfe/trunk/test/CodeGenCXX/cfi-cast.cpp
Modifie
davidxl created this revision.
davidxl added reviewers: cfe-commits, bogner, silvas, xur.
See patch-1 for description
http://reviews.llvm.org/D12716
Files:
include/clang/Driver/Options.td
include/clang/Frontend/CodeGenOptions.def
include/clang/Frontend/CodeGenOptions.h
lib/CodeGen/Backen
mzolotukhin added a comment.
Thanks, Hal, Richard!
I committed the patch with requested changes in r247107.
Michael
Repository:
rL LLVM
http://reviews.llvm.org/D12313
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org
probinson added inline comments.
Comment at: lib/CodeGen/CGDebugInfo.cpp:3263-3264
@@ +3262,4 @@
+ const NamespaceDecl *NSDecl = UD.getNominatedNamespace();
+ if (!NSDecl->isAnonymousNamespace() ||
+ CGM.getTarget().getTriple().isPS4CPU()) {
+DBuilder.createImportedMo
1 - 100 of 119 matches
Mail list logo