Author: davidxl
Date: Fri Jul 22 23:28:59 2016
New Revision: 276517
URL: http://llvm.org/viewvc/llvm-project?rev=276517&view=rev
Log:
[Profile] Use a flag to enable PGO rather than the profraw filename
Patch by Jake VanAdrighem
Differential Revision: http://reviews.llvm.org/D22608
Modified:
This revision was automatically updated to reflect the committed changes.
Closed by commit rL276517: [Profile] Use a flag to enable PGO rather than the
profraw filename (authored by davidxl).
Changed prior to commit:
https://reviews.llvm.org/D22608?vs=64812&id=65216#toc
Repository:
rL LLVM
faisalv accepted this revision.
faisalv added a reviewer: faisalv.
faisalv added a comment.
This revision is now accepted and ready to land.
Thanks Richard. Your request to separate out and centralize the diagnostic
portion of the patch certainly seems like a good idea - and I shall do so
separ
Author: faisalv
Date: Fri Jul 22 23:05:19 2016
New Revision: 276514
URL: http://llvm.org/viewvc/llvm-project?rev=276514&view=rev
Log:
[cxx1z-constexpr-lambda] Make a lambda's closure type eligible as a
literal-type in C++1z
Additionally, for pre-C++1z, instead of forbidding a lambda's closure t
Author: ericwf
Date: Fri Jul 22 22:10:56 2016
New Revision: 276511
URL: http://llvm.org/viewvc/llvm-project?rev=276511&view=rev
Log:
Implement P0392r0. Integrate filesystem::path and string_view.
Modified:
libcxx/trunk/include/experimental/filesystem
libcxx/trunk/src/experimental/filesyst
Author: rsmith
Date: Fri Jul 22 21:39:52 2016
New Revision: 276509
URL: http://llvm.org/viewvc/llvm-project?rev=276509&view=rev
Log:
Work around MSVC's lack of support for unrestricted unions by making this
struct a bit bigger under MSVC (this shouldn't be a big deal; we typically
allocate no more
Author: rsmith
Date: Fri Jul 22 21:32:21 2016
New Revision: 276508
URL: http://llvm.org/viewvc/llvm-project?rev=276508&view=rev
Log:
Add -fmodules-ts flag to cc1 for the provisional C++ modules TS, and mark
'module' and 'import' as keywords when the flag is specified.
Added:
cfe/trunk/test/Le
rjmccall added a comment.
Note that the presence of the mcount call itself will significantly impact
inlining and, really, the entire optimization pipeline. Having this be a late
pass seems more in keeping with what I assume is a goal that this
instrumentation doesn't drastically change the ge
hfinkel added a comment.
In https://reviews.llvm.org/D22666#493706, @honggyu.kim wrote:
> In https://reviews.llvm.org/D22666#493109, @rjmccall wrote:
>
> > In https://reviews.llvm.org/D22666#493026, @hfinkel wrote:
> >
> > > What's the actual desired behavior here? Should the inliner strip out
>
honggyu.kim added a comment.
In https://reviews.llvm.org/D22666#493109, @rjmccall wrote:
> In https://reviews.llvm.org/D22666#493026, @hfinkel wrote:
>
> > What's the actual desired behavior here? Should the inliner strip out calls
> > to mcount() as it inlines?
>
>
> I think they basically just
Author: ericwf
Date: Fri Jul 22 20:43:53 2016
New Revision: 276507
URL: http://llvm.org/viewvc/llvm-project?rev=276507&view=rev
Log:
Revert r276506 - Diagnose invalid memory order arguments in .
There is a bug in Clang 3.6 and earlier that causes compile failures.
I suspect it's due to the usage
ygao created this revision.
ygao added reviewers: gribozavr, aaron.ballman.
ygao added a subscriber: cfe-commits.
Hi,
I was preparing our internal release documentation with our tech writers, and
when reviewing the clang manual for the current release, we think the current
documentation of type sa
zlei created this revision.
zlei added reviewers: rmaprath, EricWF, cfe-commits.
Currently libunwind can be configured to build either shared lib or static lib,
but not both. Downstream package maintainers may want to build/install shared
lib and static lib simultaneously, without configuring tw
Author: ericwf
Date: Fri Jul 22 20:16:55 2016
New Revision: 276506
URL: http://llvm.org/viewvc/llvm-project?rev=276506&view=rev
Log:
[libcxx] Diagnose invalid memory order arguments in . Fixes PR21179.
Summary:
This patch uses the __attribute__((enable_if)) hack suggested by @rsmith to
diagnose
alexshap created this revision.
alexshap added reviewers: bkramer, jdennett.
alexshap added subscribers: compnerd, cfe-commits.
alexshap changed the visibility of this Differential Revision from "Public (No
Login Required)" to "All Users".
Code cleanup: address FIXME in
llvm/tools/clang/include/
EricWF created this revision.
EricWF added reviewers: majnemer, jfb, rsmith.
EricWF added a subscriber: cfe-commits.
Clang currently issues diagnostics if the success memory order argument is
invalid, but it doesn't check the failure ordering argument for cmpxchg atomic
operations.
This patch ad
On Jul 22, 2016 4:45 PM, "Eric Fiselier" wrote:
>
> EricWF added inline comments.
>
>
> Comment at: test/libcxx/atomics/diagnose_invalid_memory_order.fail.cpp:87
> @@ +86,3 @@
> +x.compare_exchange_weak(val1, val2, std::memory_order_release);
> +}
> +{
> --
Author: kromanova
Date: Fri Jul 22 18:49:37 2016
New Revision: 276499
URL: http://llvm.org/viewvc/llvm-project?rev=276499&view=rev
Log:
Add doxygen comments to emmintrin.h's intrinsics.
Only around 50% of the intrinsics in this file are documented now. The patches
for the rest of the intrisics i
Author: tstellar
Date: Fri Jul 22 18:45:15 2016
New Revision: 276497
URL: http://llvm.org/viewvc/llvm-project?rev=276497&view=rev
Log:
Implement cbrt builtin
This implementation was ported from the AMD builtin library
and has been tested with piglit, OpenCV, and the ocl conformance tests.
Added:
Author: tstellar
Date: Fri Jul 22 18:45:11 2016
New Revision: 276495
URL: http://llvm.org/viewvc/llvm-project?rev=276495&view=rev
Log:
geometric/floatn.inc: Add vec8 and vec16 types
Modified:
libclc/trunk/generic/include/clc/geometric/floatn.inc
Modified: libclc/trunk/generic/include/clc/geo
Author: tstellar
Date: Fri Jul 22 18:45:13 2016
New Revision: 276496
URL: http://llvm.org/viewvc/llvm-project?rev=276496&view=rev
Log:
Implement cosh builtin
This implementation was ported from the AMD builtin library
and has been tested with piglit, OpenCV, and the ocl conformance tests.
Added:
EricWF added inline comments.
Comment at: test/libcxx/atomics/diagnose_invalid_memory_order.fail.cpp:87
@@ +86,3 @@
+x.compare_exchange_weak(val1, val2, std::memory_order_release);
+}
+{
jfb wrote:
> That's not quite true: the failure ordering is a
Author: rsmith
Date: Fri Jul 22 18:36:59 2016
New Revision: 276492
URL: http://llvm.org/viewvc/llvm-project?rev=276492&view=rev
Log:
P0217R3: Parsing support and framework for AST representation of C++1z
decomposition declarations.
There are a couple of things in the wording that seem strange her
jfb added inline comments.
Comment at: test/libcxx/atomics/diagnose_invalid_memory_order.fail.cpp:87
@@ +86,3 @@
+x.compare_exchange_weak(val1, val2, std::memory_order_release);
+}
+{
That's not quite true: the failure ordering is auto-deduced from
r276489. Thanks!
On Fri, Jul 22, 2016 at 3:17 PM, Richard Smith
wrote:
> On Fri, Jul 22, 2016 at 3:15 PM, George Burgess IV <
> george.burgess...@gmail.com> wrote:
>
>> (Assuming "yes" :) )
>>
>> Richard, are you okay with this being merged into 3.9?
>>
>
> Yes, LGTM for 3.9.
>
>
>> On Fri, Jul
akyrtzi added a comment.
Thanks for making the changes; I'd recommend to go ahead and commit and we can
iterate post-commit if necessary.
Repository:
rL LLVM
https://reviews.llvm.org/D17820
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
Author: davidxl
Date: Fri Jul 22 17:25:01 2016
New Revision: 276484
URL: http://llvm.org/viewvc/llvm-project?rev=276484&view=rev
Log:
[Profile] Enable profile merging with -fprofile-generat[=]
This patch enables raw profile merging for this option which is the
new intended behavior.
Modified:
On Fri, Jul 22, 2016 at 3:15 PM, George Burgess IV <
george.burgess...@gmail.com> wrote:
> (Assuming "yes" :) )
>
> Richard, are you okay with this being merged into 3.9?
>
Yes, LGTM for 3.9.
> On Fri, Jul 22, 2016 at 9:54 AM, George Burgess IV <
> george.burgess...@gmail.com> wrote:
>
>> Error
(Assuming "yes" :) )
Richard, are you okay with this being merged into 3.9?
On Fri, Jul 22, 2016 at 9:54 AM, George Burgess IV <
george.burgess...@gmail.com> wrote:
> Error message says r272936 had the error, so it looks like it would,
> yeah. :)
>
> Would you like me to merge it to the 3.9 bran
jasonliu created this revision.
jasonliu added reviewers: mclow.lists, rsmith, hubert.reinterpretcast.
jasonliu added a subscriber: cfe-commits.
Standard 28.8 p3 indicates the 3-argument constructor have a default argument
for flag_type parameter:
basic_regex(const charT* p, size_t len, flag_type
v.g.vassilev added a subscriber: v.g.vassilev.
Comment at: include/clang/Lex/Preprocessor.h:270
@@ +269,3 @@
+ /// on the stem that is to be code completed.
+ IdentifierInfo *CodeCompletionII;
+
Can you initialize this variable in the PP's ctor?
Repository:
Author: vvassilev
Date: Fri Jul 22 16:08:24 2016
New Revision: 276473
URL: http://llvm.org/viewvc/llvm-project?rev=276473&view=rev
Log:
[modules] Teach the ASTWriter to ignore mutations coming from the ASTReader.
Processing update records (and loading a module, in general) might trigger
unexpecte
mmasten added a comment.
Thanks Michael. The tests have been updated.
Matt
https://reviews.llvm.org/D19544
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mmasten updated this revision to Diff 65152.
https://reviews.llvm.org/D19544
Files:
include/llvm/Analysis/TargetLibraryInfo.h
lib/Analysis/TargetLibraryInfo.cpp
test/Transforms/LoopVectorize/X86/svml-calls.ll
Index: lib/Analysis/TargetLibraryInfo.cpp
===
jasonliu created this revision.
jasonliu added reviewers: mclow.lists, rsmith, hubert.reinterpretcast.
jasonliu added a subscriber: cfe-commits.
In standard 28.8 p3, class basic_regex is supposed to have 'traits_type' and
'string_type' as its member types. However, they are missing from the libra
Author: vvassilev
Date: Fri Jul 22 15:49:31 2016
New Revision: 276469
URL: http://llvm.org/viewvc/llvm-project?rev=276469&view=rev
Log:
Add regression test for PR27699.
Added:
cfe/trunk/test/Modules/Inputs/PR27699/
cfe/trunk/test/Modules/Inputs/PR27699/Subdir/
cfe/trunk/test/Modules/I
erik.pilkington created this revision.
erik.pilkington added a reviewer: manmanren.
erik.pilkington added a subscriber: cfe-commits.
-Wpartial-availability currently fires on the following:
```
void f1() __attribute__((availability(macos, introduced=10.10)));
void f2() __attribute__((availability
lefticus retitled this revision from "C++1z constexpr support for array and
iterator" to "constexpr array support C++1z (P0031)".
lefticus updated the summary for this revision.
lefticus updated this revision to Diff 65143.
lefticus added a comment.
- Updated title and summary to be more accurate
EricWF updated this revision to Diff 65140.
EricWF added a comment.
Add comment about checking "stronger" orderings.
https://reviews.llvm.org/D22557
Files:
include/atomic
test/libcxx/atomics/diagnose_invalid_memory_order.fail.cpp
Index: test/libcxx/atomics/diagnose_invalid_memory_order.fai
Prazek added a comment.
In https://reviews.llvm.org/D22513#493303, @Eugene.Zelenko wrote:
> Since rule name is different in C++98/03 and C++11 or newer it will make
> sense to create two checks which will work only for their respective versions
> (of course, implementation should be shared). Or
Prazek added a comment.
I will look again soon, but it looks much better right now!
Comment at: clang-tidy/cppcoreguidelines/RuleOfFiveAndZeroCheck.cpp:43-57
@@ +42,17 @@
+
+llvm::StringRef RuleOfFiveAndZeroCheck::toString(
+RuleOfFiveAndZeroCheck::SpecialMemberFunctionKind
EricWF added a reviewer: jfb.
EricWF marked an inline comment as done.
EricWF added a comment.
Address inline comments from @jfb and add him as a reviewer.
Comment at: test/libcxx/atomics/diagnose_invalid_memory_order.fail.cpp:86
@@ +85,3 @@
+// does not generate any dia
Eugene.Zelenko added a comment.
Since rule name is different in C++98/03 and C++11 or newer it will make sense
to create two checks which will work only for their respective versions (of
course, implementation should be shared). Or create alias name.
Check is still not mentioned in docs/Releas
Prazek marked 3 inline comments as done.
Comment at: test/clang-tidy/boost-use-to-string.cpp:154
@@ +153,3 @@
+ float floating;
+ Fields* wierd;
+ const int &getConstInteger() const {return integer;}
alexfh wrote:
> alexfh wrote:
> > "wierd" is weird ;)
> I sho
yaxunl accepted this revision.
yaxunl added a comment.
This revision is now accepted and ready to land.
LGTM. Thanks!
Repository:
rL LLVM
https://reviews.llvm.org/D22637
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.or
Author: ericwf
Date: Fri Jul 22 15:04:56 2016
New Revision: 276462
URL: http://llvm.org/viewvc/llvm-project?rev=276462&view=rev
Log:
Merging r276215:
r276215 | ericwf | 2016-07-20 17:56:42 -0600 (Wed, 20 Jul 2016) | 9 lines
v.g.vassilev added a comment.
Could we stress test the implementation by running on files from llvm's/clang's
test suite. For example, for a test TST we could do something like: cat TST >>
tmp_TST; cat TST >> tmp_TST and run our our copy-paste checker on tmp_TST.
This way we know how many clon
ashi1 updated this revision to Diff 65126.
ashi1 marked an inline comment as done.
ashi1 added a comment.
Revised based on Sam's comments.
Repository:
rL LLVM
https://reviews.llvm.org/D22637
Files:
include/clang/Basic/OpenCLExtensions.def
lib/Basic/Targets.cpp
test/Misc/amdgcn.language
sfantao updated this revision to Diff 65123.
sfantao added a comment.
- Use the correct enumerator for return pointer, as specified in the design
document.
https://reviews.llvm.org/D22691
Files:
include/clang/AST/OpenMPClause.h
lib/AST/OpenMPClause.cpp
lib/CodeGen/CGOpenMPRuntime.cpp
l
Most of these have been broken like this for a long time so I'm not sure.
This patch as is depends on r276249 too since we were missing qualifiers in
the intrinsics file.
~Craig
On Fri, Jul 22, 2016 at 6:55 AM, Hans Wennborg wrote:
> Should we merge this to 3.9?
>
> On Thu, Jul 21, 2016 at 3:38
v.g.vassilev accepted this revision.
v.g.vassilev added a comment.
This revision is now accepted and ready to land.
LGTM, given the comments are addressed.
Comment at: include/clang/Analysis/CloneDetection.h:37
@@ +36,3 @@
+ /// Stmt, then S is a pointer to this Stmt.
+ Stmt c
mgehre added a comment.
Friendly ping
https://reviews.llvm.org/D15031
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
yaxunl added inline comments.
Comment at: test/Misc/amdgcn.languageOptsOpenCL.cl:190
@@ +189,3 @@
+// expected-warning@-1{{unsupported OpenCL extension 'cl_khr_mipmap_image' -
ignoring}}
+#endif
+
we also need to check the else case for `__OPENCL_C_VERSION__ < 20
This revision was automatically updated to reflect the committed changes.
Closed by commit rL276455: Add .rgba syntax extension to ext_vector_type types
(authored by pirama).
Changed prior to commit:
https://reviews.llvm.org/D20602?vs=62305&id=65119#toc
Repository:
rL LLVM
https://reviews.l
Author: pirama
Date: Fri Jul 22 13:49:43 2016
New Revision: 276455
URL: http://llvm.org/viewvc/llvm-project?rev=276455&view=rev
Log:
Add .rgba syntax extension to ext_vector_type types
Summary:
This patch enables .rgba accessors to ext_vector_type types and adds
tests for syntax validation and co
ashi1 updated this revision to Diff 65118.
ashi1 marked an inline comment as done.
ashi1 added a comment.
Revised based on Sam's comments. Added to tests
Misc\amdgcn.languageOptsOpenCL.cl and SemaOpenCL\extension-version.cl
Repository:
rL LLVM
https://reviews.llvm.org/D22637
Files:
includ
manmanren added a comment.
In https://reviews.llvm.org/D22636#493072, @benlangmuir wrote:
> > Can you point me to the source codes where we use rename to replace the
> > file? I am curious on how this all works out.
>
>
> This is the same as clang's handling of other output files. See
> {{Comp
rjmccall added a comment.
In https://reviews.llvm.org/D22666#493026, @hfinkel wrote:
> What's the actual desired behavior here? Should the inliner strip out calls
> to mcount() as it inlines?
I think they basically just want a late pass (as in immediately prior to
codegen) to insert the mcoun
sfantao created this revision.
sfantao added reviewers: ABataev, hfinkel, carlo.bertolli, arpith-jacob, kkwli0.
sfantao added subscribers: cfe-commits, caomhin.
This patch adds support for the use_device_ptr clause. It includes changes in
SEMA that could not be tested without codegen, namely, the
sfantao created this revision.
sfantao added reviewers: ABataev, hfinkel, carlo.bertolli, kkwli0.
sfantao added subscribers: cfe-commits, caomhin.
This patch fixes a bug in the map of array sections whose base is a reference
to a pointer. The existing mapping support was not prepared to deal with
Author: annat
Date: Fri Jul 22 12:50:08 2016
New Revision: 276448
URL: http://llvm.org/viewvc/llvm-project?rev=276448&view=rev
Log:
Clang changes for overloading invariant.start and end intrinsics
This change depends on the corresponding LLVM change at:
https://reviews.llvm.org/D22519
The llvm.i
benlangmuir added a comment.
> Can you point me to the source codes where we use rename to replace the file?
> I am curious on how this all works out.
This is the same as clang's handling of other output files. See
{{CompilerInstance::createOutputFile}} and {{clearOutputFiles}}. When we are
weimingz added a comment.
LGTM.
Maybe @renato can review it too.
Thanks Oliver and Saleem.
https://reviews.llvm.org/D22292
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: chapuni
Date: Fri Jul 22 12:33:04 2016
New Revision: 276445
URL: http://llvm.org/viewvc/llvm-project?rev=276445&view=rev
Log:
ClangRenameTests: Update libdeps.
Modified:
clang-tools-extra/trunk/unittests/clang-rename/CMakeLists.txt
Modified: clang-tools-extra/trunk/unittests/clang-re
zaks.anna added inline comments.
Comment at: test/Analysis/PR12421.c:11
@@ +10,2 @@
+
+// CHECK: warning: Path diagnostic report is not generated. HTMLDiagnostics
does not support diagnostics that cross file boundaries.
We should use the name of the diagnostic co
Thanks Ranjeet,
Indeed I was planning to update the patches to top of trunk and reload them, so
if you can save me this trouble I would appreciate that.
Once we upload the updated patches I would like to ping the community and ask
for feedback.
Thanks,
Amjad
> -Original Message-
> From:
Author: jvesely
Date: Fri Jul 22 12:24:24 2016
New Revision: 276443
URL: http://llvm.org/viewvc/llvm-project?rev=276443&view=rev
Log:
AMDGPU: Implement get_global_offset builtin
Also fix get_global_id to consider offset
No idea how to add this for ptx, so they are stuck with the old get_global_id
Author: jvesely
Date: Fri Jul 22 12:24:20 2016
New Revision: 276442
URL: http://llvm.org/viewvc/llvm-project?rev=276442&view=rev
Log:
AMDGPU: Use clang intrinsics for workitem builtins
v2: split into 2 patches
use clang builtins for other intrinsics as well
v3: Fix warnings
Switch r600 t
Eugene.Zelenko updated this revision to Diff 65099.
Eugene.Zelenko added a comment.
Don't include DiagnosticIDs.h.
Repository:
rL LLVM
https://reviews.llvm.org/D22656
Files:
clang-apply-replacements/tool/ClangApplyReplacementsMain.cpp
Index: clang-apply-replacements/tool/ClangApplyReplace
rs added a subscriber: rs.
rs added a comment.
Hi Amjad,
are you still planning on getting this patch and
https://reviews.llvm.org/D16077 committed ? It looks like these two patches are
final pieces in the puzzle to get macro information in the DWARF debug output.
I've downloaded the diffs and
hfinkel added a subscriber: hfinkel.
hfinkel added a comment.
What's the actual desired behavior here? Should the inliner strip out calls to
mcount() as it inlines?
https://reviews.llvm.org/D22666
___
cfe-commits mailing list
cfe-commits@lists.llvm
jbcoe updated this revision to Diff 65096.
jbcoe marked 6 inline comments as done.
jbcoe added a comment.
Address comments from review, thanks for those.
https://reviews.llvm.org/D22513
Files:
clang-tidy/cppcoreguidelines/CMakeLists.txt
clang-tidy/cppcoreguidelines/CppCoreGuidelinesTidyModu
Eugene.Zelenko added a comment.
It's used for DiagnosticIDs.. However, in https://reviews.llvm.org/D22656
Alexander told that DiagnosticIDs.h should be treated as part of Diagnostic.h.
Repository:
rL LLVM
https://reviews.llvm.org/D22654
___
cfe-
john.brawn added a comment.
Ping.
Repository:
rL LLVM
https://reviews.llvm.org/D21385
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
john.brawn added a comment.
Ping.
Repository:
rL LLVM
https://reviews.llvm.org/D1
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Error message says r272936 had the error, so it looks like it would, yeah.
:)
Would you like me to merge it to the 3.9 branch?
On Fri, Jul 22, 2016 at 6:51 AM, Hans Wennborg wrote:
> Does this apply to 3.9?
>
> On Wed, Jul 20, 2016 at 11:28 PM, George Burgess IV via cfe-commits
> wrote:
> > Au
jbcoe removed rL LLVM as the repository for this revision.
jbcoe updated this revision to Diff 65092.
jbcoe marked 2 inline comments as done.
jbcoe added a comment.
Addressed review comments, thanks for those, check is nicer now.
https://reviews.llvm.org/D22513
Files:
clang-tidy/cppcoreguidel
Author: annat
Date: Fri Jul 22 10:37:56 2016
New Revision: 276425
URL: http://llvm.org/viewvc/llvm-project?rev=276425&view=rev
Log:
test commit. update comment grammatically. NFC
Modified:
cfe/trunk/lib/CodeGen/CGDeclCXX.cpp
Modified: cfe/trunk/lib/CodeGen/CGDeclCXX.cpp
URL:
http://llvm.org
zaks.anna added a comment.
Great!
How does this work when a path spans multiple files, specifically, when the
definitions from a header file are inlined? We could add file names but only in
cases the file name does not match the main file.
> and hard to navigate (especially when using a specia
teemperor updated this revision to Diff 65086.
teemperor added a comment.
- StmtDataCollector is now using ConstStmtVisitor
- Added tests for StmtDataCollector
https://reviews.llvm.org/D22514
Files:
lib/Analysis/CloneDetection.cpp
test/Analysis/copypaste/test-asm.cpp
test/Analysis/copypas
zaks.anna accepted this revision.
zaks.anna added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D22671
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo
Author: emaste
Date: Fri Jul 22 10:00:42 2016
New Revision: 276424
URL: http://llvm.org/viewvc/llvm-project?rev=276424&view=rev
Log:
Merge r276128: libunwind: limit stack usage in unwind cursor
Modified:
libunwind/branches/release_39/ (props changed)
libunwind/branches/release_39/includ
Author: jvesely
Date: Fri Jul 22 10:00:08 2016
New Revision: 276423
URL: http://llvm.org/viewvc/llvm-project?rev=276423&view=rev
Log:
ptx: Fix builtin names after clang r274770
Signed-off-by: Jan Vesely
Acked-By: Aaron Watry
Modified:
libclc/trunk/ptx-nvidiacl/lib/synchronization/barrier.c
alexfh requested changes to this revision.
alexfh added a comment.
This revision now requires changes to proceed.
In many cases a transitively included header can be considered a part of a
public interface of an already included header, e.g. it's reasonable to assume
that DiagnosticIDs.h is a pa
sdardis added a comment.
Actually, seeing your reply on the related patch, keep the comment as it
reflects trunk.
https://reviews.llvm.org/D22679
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo
sdardis added a comment.
Yes, I think you should also reflect mpf's comment here too.
https://reviews.llvm.org/D22679
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sdardis accepted this revision.
sdardis added a comment.
This revision is now accepted and ready to land.
LGTM.
https://reviews.llvm.org/D22679
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cf
Alexander_Droste updated this revision to Diff 65074.
Alexander_Droste added a comment.
- fix typo
https://reviews.llvm.org/D21962
Files:
clang-tidy/CMakeLists.txt
clang-tidy/mpi/CMakeLists.txt
clang-tidy/mpi/MPITidyModule.cpp
clang-tidy/mpi/TypeMismatchCheck.cpp
clang-tidy/mpi/TypeMi
dsanders created this revision.
dsanders added a reviewer: sdardis.
dsanders added a subscriber: cfe-commits.
Unfortunately we can't enable it for all N64 because it is not yet possible to
distinguish N32 from N64 from the triple on other environments.
Author: rksimon
Date: Fri Jul 22 08:58:56 2016
New Revision: 276417
URL: http://llvm.org/viewvc/llvm-project?rev=276417&view=rev
Log:
[X86][AVX] Added support for lowering to VBROADCASTF128/VBROADCASTI128 with
generic IR
As discussed on D22460, I've updated the vbroadcastf128 pd256/ps256 builtin
Alexander_Droste updated this revision to Diff 65072.
Alexander_Droste added a comment.
This update addresses all requested changes. All integration tests are still
passing.
https://reviews.llvm.org/D21962
Files:
clang-tidy/CMakeLists.txt
clang-tidy/mpi/CMakeLists.txt
clang-tidy/mpi/MPIT
Alexander_Droste marked 5 inline comments as done.
Alexander_Droste added a comment.
> You need to add this to ClangTidyMain.cpp:
Thanks for pointing this out and reviewing the code once more!
https://reviews.llvm.org/D21962
___
cfe-commits mailin
Should we merge this to 3.9?
On Thu, Jul 21, 2016 at 3:38 AM, Craig Topper via cfe-commits
wrote:
> Author: ctopper
> Date: Thu Jul 21 02:38:43 2016
> New Revision: 276250
>
> URL: http://llvm.org/viewvc/llvm-project?rev=276250&view=rev
> Log:
> [Sema,X86] Add explicit check to ensure that builti
Richard: should we merge this to 3.9?
On Thu, Jul 21, 2016 at 7:03 PM, David Majnemer via cfe-commits
wrote:
> Author: majnemer
> Date: Thu Jul 21 18:03:43 2016
> New Revision: 276352
>
> URL: http://llvm.org/viewvc/llvm-project?rev=276352&view=rev
> Log:
> [Sema] Handle errors during rewriteBuil
Richard: should we merge this to 3.9?
On Thu, Jul 21, 2016 at 6:31 PM, Erik Pilkington via cfe-commits
wrote:
> Author: epilk
> Date: Thu Jul 21 17:31:40 2016
> New Revision: 276350
>
> URL: http://llvm.org/viewvc/llvm-project?rev=276350&view=rev
> Log:
> [CodeGen] Fix a crash when constant foldi
pipping added a subscriber: pipping.
pipping added a comment.
Any updates on this?
https://reviews.llvm.org/D5409
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: courbet
Date: Fri Jul 22 08:45:00 2016
New Revision: 276415
URL: http://llvm.org/viewvc/llvm-project?rev=276415&view=rev
Log:
Revert "Revert "[clang-tidy] new cppcoreguidelines-slicing""
Second try for r276408
Added:
clang-tools-extra/trunk/clang-tidy/cppcoreguidelines/SlicingCheck.c
Does this apply to 3.9?
On Wed, Jul 20, 2016 at 11:28 PM, George Burgess IV via cfe-commits
wrote:
> Author: gbiv
> Date: Wed Jul 20 22:28:13 2016
> New Revision: 276232
>
> URL: http://llvm.org/viewvc/llvm-project?rev=276232&view=rev
> Log:
> [Sema] Fix PR28623.
>
> In atomic builtins, we assume
This revision was automatically updated to reflect the committed changes.
Closed by commit rL276414: [clang-rename] introduce better symbol finding
(authored by omtcyfz).
Changed prior to commit:
https://reviews.llvm.org/D22465?vs=65066&id=65067#toc
Repository:
rL LLVM
https://reviews.llvm.
Author: omtcyfz
Date: Fri Jul 22 08:41:09 2016
New Revision: 276414
URL: http://llvm.org/viewvc/llvm-project?rev=276414&view=rev
Log:
[clang-rename] introduce better symbol finding
This patch introduces:
* TypeLoc visiting, which helps a lot in renaming types
* NestedNameSpecifierLoc visiting (t
1 - 100 of 144 matches
Mail list logo
