dberris created this revision.
dberris added reviewers: echristo, rafael, grosbach.
dberris added a subscriber: cfe-commits.
Herald added a subscriber: mehdi_amini.
Remove the -mrelax-all options passed onto the integrated assembler in
-O0.
http://reviews.llvm.org/D21830
Files:
lib/Driver/Tool
Author: ibreger
Date: Wed Jun 29 03:14:17 2016
New Revision: 274110
URL: http://llvm.org/viewvc/llvm-project?rev=274110&view=rev
Log:
[AVX512] Zero extend cmp intrinsic return value.
Differential Revision: http://reviews.llvm.org/D21746
Modified:
cfe/trunk/lib/CodeGen/CGBuiltin.cpp
cfe/
This revision was automatically updated to reflect the committed changes.
Closed by commit rL274110: [AVX512] Zero extend cmp intrinsic return value.
(authored by ibreger).
Changed prior to commit:
http://reviews.llvm.org/D21746?vs=61953&id=62185#toc
Repository:
rL LLVM
http://reviews.llvm
hokein created this revision.
hokein added a reviewer: alexfh.
hokein added subscribers: cfe-commits, Eugene.Zelenko.
Fix PR28350.
http://reviews.llvm.org/D21833
Files:
clang-tidy/misc/UnusedUsingDeclsCheck.cpp
test/clang-tidy/misc-unused-using-decls.cpp
Index: test/clang-tidy/misc-unused-u
This revision was automatically updated to reflect the committed changes.
Closed by commit rL274114: [Driver][AArch64] Add support for Broadcom Vulcan
core. (authored by pgode).
Changed prior to commit:
http://reviews.llvm.org/D21501?vs=61192&id=62195#toc
Repository:
rL LLVM
http://reviews.
Author: pgode
Date: Wed Jun 29 05:00:31 2016
New Revision: 274114
URL: http://llvm.org/viewvc/llvm-project?rev=274114&view=rev
Log:
[Driver][AArch64] Add support for Broadcom Vulcan core.
Adding support for new Broadcom Vulcan core (ARMv8.1A).
Differential Revision: http://reviews.llvm.org/D2150
nhaustov updated this revision to Diff 62198.
nhaustov added a comment.
Update after discussion with Sam and review from Anastasia.
http://reviews.llvm.org/D21367
Files:
include/clang/Basic/Specifiers.h
lib/AST/ItaniumMangle.cpp
lib/AST/Type.cpp
lib/AST/TypePrinter.cpp
lib/Basic/Targe
This revision was automatically updated to reflect the committed changes.
Closed by commit rL274119: Use ArgList::hasFlag to check if -miamcu/-mno-iamcu
is passed. NFC. (authored by aturetsk).
Changed prior to commit:
http://reviews.llvm.org/D21641?vs=61662&id=62203#toc
Repository:
rL LLVM
Author: aturetsk
Date: Wed Jun 29 05:57:17 2016
New Revision: 274119
URL: http://llvm.org/viewvc/llvm-project?rev=274119&view=rev
Log:
Use ArgList::hasFlag to check if -miamcu/-mno-iamcu is passed. NFC.
Differential Revision: http://reviews.llvm.org/D21641
Modified:
cfe/trunk/lib/Driver/Driv
Author: bader
Date: Wed Jun 29 07:25:58 2016
New Revision: 274120
URL: http://llvm.org/viewvc/llvm-project?rev=274120&view=rev
Log:
[OpenCL] Fix typo in as_type test.
Reset astype variable in f6 function to avoid matching with wrong value from f5
function.
Modified:
cfe/trunk/test/CodeGenOp
Author: bader
Date: Wed Jun 29 07:30:26 2016
New Revision: 274122
URL: http://llvm.org/viewvc/llvm-project?rev=274122&view=rev
Log:
[OpenCL] Add attribute 'pure' to read_image built-in functions to enable
optimizations.
Reviewers: Anastasia, yaxunl
Subscribers: pekka.jaaskelainen, pxli168, cfe-
This revision was automatically updated to reflect the committed changes.
Closed by commit rL274122: [OpenCL] Add attribute 'pure' to read_image built-in
functions to enable… (authored by bader).
Changed prior to commit:
http://reviews.llvm.org/D21795?vs=62094&id=62210#toc
Repository:
rL LLV
mclow.lists accepted this revision.
mclow.lists added a comment.
This revision is now accepted and ready to land.
This looks fine to me.
http://reviews.llvm.org/D20334
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi
m_zuckerman created this revision.
m_zuckerman added reviewers: AsafBadouh, igorb, delena, craig.topper.
m_zuckerman added a subscriber: cfe-commits.
By SPEC, instruction vpermilps gets IMM bigger than 15.
Below, you can see that IMM can be any number between 0 to 255 include.
SELECT4(src, con
igorb accepted this revision.
igorb added a comment.
This revision is now accepted and ready to land.
LGTM
http://reviews.llvm.org/D21836
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm
tavianator updated this revision to Diff 62218.
tavianator added a comment.
Added error handling for pthread_key uses
http://reviews.llvm.org/D21803
Files:
cmake/config-ix.cmake
src/CMakeLists.txt
src/cxa_thread_atexit.cpp
test/CMakeLists.txt
test/cxa_thread_atexit_test.pass.cpp
tes
bcraig added subscribers: rmaprath, bcraig.
bcraig added a comment.
You should look at __thread_specific_ptr in libcxx's . It does a lot
of these things in order to satisfy the requirements of
notify_all_at_thread_exit, set_value_at_thread_exit, and
make_ready_at_thread_exit.
@rmaprath has be
pgousseau added a comment.
Ping!
http://reviews.llvm.org/D20867
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
DmitryPolukhin updated this revision to Diff 62213.
DmitryPolukhin marked 2 inline comments as done.
DmitryPolukhin added a comment.
Use SmallVector instead of SmallSetVector and sort the vector when needed.
http://reviews.llvm.org/D18035
Files:
lib/AST/ItaniumMangle.cpp
lib/Sema/SemaDeclAt
lihuang added a comment.
Hi Adam,
The change in http://reviews.llvm.org/D18777 breaks this test becasue it
converts some sexts to zexts, which cannot be eliminated by
indvar-simplification after widening IV.
The IR after indvar-simplification and before loop-vectorization is like:
...
%in
Author: rksimon
Date: Wed Jun 29 09:04:08 2016
New Revision: 274126
URL: http://llvm.org/viewvc/llvm-project?rev=274126&view=rev
Log:
[X86][SSE2] Updated tests to match
llvm\test\CodeGen\X86\sse2-intrinsics-fast-isel-x86_64.ll
Modified:
cfe/trunk/test/CodeGen/sse2-builtins.c
Modified: cfe/t
aaron.ballman added a subscriber: aaron.ballman.
Comment at: clang-tidy/misc/UnusedUsingDeclsCheck.cpp:22
@@ +21,3 @@
+namespace {
+// FIXME: Move this node matcher to ASTMatcher.
+const internal::VariadicDynCastAllOfMatcher enumType;
Can you fix this fixme as a p
kmensah added a comment.
Another friendly Ping
Repository:
rL LLVM
http://reviews.llvm.org/D21510
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
On Tue, Jun 28, 2016 at 4:55 PM, Manman Ren via cfe-commits
wrote:
> Author: mren
> Date: Tue Jun 28 15:55:30 2016
> New Revision: 274064
>
> URL: http://llvm.org/viewvc/llvm-project?rev=274064&view=rev
> Log:
> AvailabilityAttr: we accept "macos" as the platform name.
>
> We continue accepting "m
We need to benchmark this to see if it still makes a difference.
My canonical test is the .s of a lto of clang.
Cheers,
Rafael
On 29 June 2016 at 03:37, Dean Michael Berris wrote:
> dberris created this revision.
> dberris added reviewers: echristo, rafael, grosbach.
> dberris added a subscrib
Author: ahatanak
Date: Wed Jun 29 10:26:13 2016
New Revision: 274132
URL: http://llvm.org/viewvc/llvm-project?rev=274132&view=rev
Log:
[libcxx] Fix a bug in strstreambuf::overflow.
The end pointer should point to one past the end of the newly allocated
buffer.
rdar://problem/24265174
Differenti
This revision was automatically updated to reflect the committed changes.
Closed by commit rL274132: [libcxx] Fix a bug in strstreambuf::overflow.
(authored by ahatanak).
Changed prior to commit:
http://reviews.llvm.org/D20334?vs=58740&id=62225#toc
Repository:
rL LLVM
http://reviews.llvm.or
Good point! In r274133.
Manman
On Wed, Jun 29, 2016 at 7:30 AM, Aaron Ballman
wrote:
> On Tue, Jun 28, 2016 at 4:55 PM, Manman Ren via cfe-commits
> wrote:
> > Author: mren
> > Date: Tue Jun 28 15:55:30 2016
> > New Revision: 274064
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=274064&v
Author: mren
Date: Wed Jun 29 10:50:08 2016
New Revision: 274133
URL: http://llvm.org/viewvc/llvm-project?rev=274133&view=rev
Log:
AvailabilityAttr: update the document for macosx
Modified:
cfe/trunk/include/clang/Basic/AttrDocs.td
Modified: cfe/trunk/include/clang/Basic/AttrDocs.td
URL:
ht
tavianator added a comment.
In http://reviews.llvm.org/D21803#469988, @bcraig wrote:
> You should look at __thread_specific_ptr in libcxx's . It does a lot
> of these things in order to satisfy the requirements of
> notify_all_at_thread_exit, set_value_at_thread_exit, and
> make_ready_at_thre
majnemer added a subscriber: majnemer.
majnemer added a comment.
Why are you making this change? I cannot see a justification in the description.
http://reviews.llvm.org/D21830
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llv
sfantao updated this revision to Diff 62230.
sfantao added a comment.
- Better organize how the offload action is used and add more comments to
document what is going on.
- Rebase.
http://reviews.llvm.org/D18171
Files:
include/clang/Driver/Action.h
include/clang/Driver/Compilation.h
incl
sfantao added a comment.
@tra, any other comments/suggestions about this patch?
Thanks!
Samuel
http://reviews.llvm.org/D18171
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sfantao updated this revision to Diff 62231.
sfantao added a comment.
- Rebase.
http://reviews.llvm.org/D18172
Files:
include/clang/Driver/Compilation.h
lib/Driver/Driver.cpp
Index: lib/Driver/Driver.cpp
===
--- lib/Driver/Dri
sfantao created this revision.
sfantao added reviewers: echristo, tra, jlebar, hfinkel, ABataev, rsmith.
sfantao added subscribers: carlo.bertolli, arpith-jacob, caomhin,
andreybokhanko, Hahnfeld, cfe-commits.
Herald added a subscriber: mehdi_amini.
This creates a tool selector in the driver that
bcraig added a comment.
In http://reviews.llvm.org/D21803#470060, @tavianator wrote:
> In http://reviews.llvm.org/D21803#469988, @bcraig wrote:
>
> > You should look at __thread_specific_ptr in libcxx's . It does a
> > lot of these things in order to satisfy the requirements of
> > notify_all_
rnk accepted this revision.
rnk added a comment.
lgtm
Cute, preserves debug info compatibility at the cost of some complexity.
http://reviews.llvm.org/D21783
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mail
rnk added a comment.
After writing r266366, we discovered that GCC accepts none of the code in that
test case, so we should consider turning -flax-vector-conversions off by
default.
Comment at: test/Sema/vector-cast.c:65
@@ -59,1 +64,3 @@
+ d = f2;
+ d = d + f2;
}
-
yaxunl accepted this revision.
yaxunl added a comment.
This revision is now accepted and ready to land.
LGTM. Thanks!
http://reviews.llvm.org/D21367
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listin
sfantao created this revision.
sfantao added reviewers: echristo, tra, jlebar, hfinkel, ABataev, rsmith.
sfantao added subscribers: caomhin, carlo.bertolli, arpith-jacob, Hahnfeld,
cfe-commits.
Herald added a subscriber: mehdi_amini.
This patch adds new logic to create the necessary tool chains t
sfantao created this revision.
sfantao added reviewers: echristo, tra, jlebar, hfinkel, ABataev, rsmith.
sfantao added subscribers: caomhin, carlo.bertolli, arpith-jacob,
andreybokhanko, Hahnfeld, cfe-commits.
Herald added a subscriber: mehdi_amini.
This patch adds a new specialized action builde
pirama added a comment.
Ping...
http://reviews.llvm.org/D21334
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
pirama added a comment.
Ping...
http://reviews.llvm.org/D20602
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rmaprath added a comment.
In http://reviews.llvm.org/D21803#470060, @tavianator wrote:
> In http://reviews.llvm.org/D21803#469988, @bcraig wrote:
>
> > @rmaprath has been doing some work to make the threading runtime library
> > swappable. I don't recall if his work extended to libcxxabi or not
sfantao created this revision.
sfantao added reviewers: echristo, tra, jlebar, hfinkel, ABataev, rsmith.
sfantao added subscribers: caomhin, carlo.bertolli, arpith-jacob,
andreybokhanko, Hahnfeld, cfe-commits.
Herald added a subscriber: mehdi_amini.
This patch adds logic to create jobs for OpenMP
majnemer added inline comments.
Comment at: src/cxa_thread_atexit.cpp:64-90
@@ -18,8 +63,29 @@
+_LIBCXXABI_FUNC_VIS int __cxa_thread_atexit(Dtor dtor, void *obj,
void *dso_symbol) throw() {
- extern int __cxa_thread_atexit_impl(void (*
sfantao created this revision.
sfantao added reviewers: echristo, tra, jlebar, hfinkel, ABataev, rsmith.
sfantao added subscribers: caomhin, carlo.bertolli, arpith-jacob,
andreybokhanko, Hahnfeld, cfe-commits.
Herald added a subscriber: mehdi_amini.
This patch includes support for argument transl
sfantao updated this revision to Diff 62249.
sfantao added a comment.
Herald added a subscriber: mehdi_amini.
- Reorganize file handlers. Generate empty files if no components exist in the
bundle instead of just failing.
http://reviews.llvm.org/D13909
Files:
test/Driver/clang-offload-bundler
AntonBikineev added a subscriber: cfe-commits.
AntonBikineev updated this revision to Diff 62254.
http://reviews.llvm.org/D21834
Files:
include/clang/AST/Stmt.h
include/clang/Parse/Parser.h
include/clang/Sema/Sema.h
lib/AST/ASTImporter.cpp
lib/AST/Stmt.cpp
lib/Analysis/BodyFarm.cpp
rnk accepted this revision.
rnk added a comment.
lgtm
http://reviews.llvm.org/D21666
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sfantao created this revision.
sfantao added reviewers: echristo, tra, jlebar, hfinkel, ABataev.
sfantao added subscribers: caomhin, carlo.bertolli, arpith-jacob, Hahnfeld,
andreybokhanko, cfe-commits.
Herald added a subscriber: mehdi_amini.
This patch adds the capability to bundle object files i
Author: rnk
Date: Wed Jun 29 13:29:21 2016
New Revision: 274147
URL: http://llvm.org/viewvc/llvm-project?rev=274147&view=rev
Log:
Re-land "[MS] Don't expect vftables to be provided for extern template
instantiations"
Reverts r273305 and re-instates r273296.
We needed to fix a bug in Sema::MarkV
sfantao created this revision.
sfantao added reviewers: echristo, tra, jlebar, hfinkel, ABataev.
sfantao added subscribers: caomhin, carlo.bertolli, arpith-jacob,
andreybokhanko, Hahnfeld, cfe-commits.
Herald added a subscriber: mehdi_amini.
In order to save the user from dealing with multiple ou
sfantao created this revision.
sfantao added reviewers: echristo, tra, jlebar, hfinkel, ABataev.
sfantao added subscribers: caomhin, carlo.bertolli, arpith-jacob,
andreybokhanko, Hahnfeld, cfe-commits.
Herald added a subscriber: mehdi_amini.
Each time that offloading support is requested by the u
sfantao created this revision.
sfantao added reviewers: echristo, tra, jlebar, hfinkel, ABataev.
sfantao added subscribers: caomhin, carlo.bertolli, arpith-jacob,
andreybokhanko, Hahnfeld, cfe-commits.
Herald added a subscriber: mehdi_amini.
This patch adds the support to create a job for the `Of
Author: yrnkrn
Date: Wed Jun 29 13:55:53 2016
New Revision: 274149
URL: http://llvm.org/viewvc/llvm-project?rev=274149&view=rev
Log:
Add CapturedDecl::parameters() ArrayRef accessors following r273647.
Modified:
cfe/trunk/include/clang/AST/Decl.h
Modified: cfe/trunk/include/clang/AST/Decl.h
sfantao created this revision.
sfantao added reviewers: echristo, tra, jlebar, hfinkel, ABataev.
sfantao added subscribers: caomhin, carlo.bertolli, arpith-jacob,
andreybokhanko, Hahnfeld, cfe-commits.
Herald added a subscriber: mehdi_amini.
This patch adds the support to create jobs for the `Off
Hi all!
I'd like to establish a policy for Clang's default language standard (if
none is specified with -std), as follows:
Clang defaults to the most recent published standard for the selected
language that it fully implements.
The practical impact of this is that clang++ will default to C++14
For clang-cl, we should follow cl's model of selecting a standard. Before
C++17, this was "use newest language the compiler knows about" (keyed off
-fmsc-version; clang-cl detects the system msvc version by default if
that's not passed in). After C++17, there's an explicit language flag, and
we use
On Wed, Jun 29, 2016 at 12:14 PM, Nico Weber wrote:
> For clang-cl, we should follow cl's model of selecting a standard. Before
> C++17, this was "use newest language the compiler knows about" (keyed off
> -fmsc-version; clang-cl detects the system msvc version by default if
> that's not passed i
On Wed, Jun 29, 2016 at 12:09 PM, Richard Smith via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Hi all!
>
> I'd like to establish a policy for Clang's default language standard (if
> none is specified with -std), as follows:
>
> Clang defaults to the most recent published standard for the
tavianator added a comment.
In http://reviews.llvm.org/D21803#470086, @bcraig wrote:
> It also intentionally leaks the pthread key. Does the __thread_specific_ptr
> rationale hold for this change as well?
Hmm, maybe? If other global destructors run after ~DtorListHolder(), and they
cause a
This revision was automatically updated to reflect the committed changes.
Closed by commit rL274150: [OpenCL] Allow -cl-std and other standard -cl-
options in driver (authored by yaxunl).
Changed prior to commit:
http://reviews.llvm.org/D21031?vs=61574&id=62267#toc
Repository:
rL LLVM
http:
Author: yaxunl
Date: Wed Jun 29 14:39:32 2016
New Revision: 274150
URL: http://llvm.org/viewvc/llvm-project?rev=274150&view=rev
Log:
[OpenCL] Allow -cl-std and other standard -cl- options in driver
Allow -cl-std and other standard -cl- options from cc1 to driver.
Added a test for the options mov
On Wed, Jun 29, 2016 at 3:25 PM, Richard Smith
wrote:
> On Wed, Jun 29, 2016 at 12:14 PM, Nico Weber wrote:
>
>> For clang-cl, we should follow cl's model of selecting a standard. Before
>> C++17, this was "use newest language the compiler knows about" (keyed off
>> -fmsc-version; clang-cl detec
rnk added inline comments.
Comment at: lib/CodeGen/CGDebugInfo.cpp:1075
@@ +1074,3 @@
+ } else if (const auto *nestedRec = dyn_cast(I)) {
+if (!nestedRec->isImplicit())
+ CollectRecordNestedRecord(nestedRec, elements);
I would add `&& nestedR
On Wed, Jun 29, 2016 at 12:30 PM, Sean Silva wrote:
>
>
> On Wed, Jun 29, 2016 at 12:09 PM, Richard Smith via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
>
>> Hi all!
>>
>> I'd like to establish a policy for Clang's default language standard (if
>> none is specified with -std), as follows:
On Wed, Jun 29, 2016 at 12:47 PM Nico Weber via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> On Wed, Jun 29, 2016 at 3:25 PM, Richard Smith
> wrote:
>
>> Note that at this point I'd like to mainly focus on what our policy
>> should be;
>>
>
> Fair enough. I don't have a strong opinion on th
- Original Message -
> From: "Richard Smith via cfe-commits"
> To: "cfe-commits" , "Clang Dev"
>
> Sent: Wednesday, June 29, 2016 2:09:37 PM
> Subject: RFC: Default language standard mode policy
> Hi all!
> I'd like to establish a policy for Clang's default language standard
> (if none
On Wed, Jun 29, 2016 at 12:55 PM, Hal Finkel via cfe-dev <
cfe-...@lists.llvm.org> wrote:
>
> --
>
> *From: *"Richard Smith via cfe-commits"
> *To: *"cfe-commits" , "Clang Dev" <
> cfe-...@lists.llvm.org>
> *Sent: *Wednesday, June 29, 2016 2:09:37 PM
> *Subject: *RFC:
bcraig added a comment.
> Hmm, maybe? If other global destructors run after ~DtorListHolder(), and
> they cause a thread_local to be initialized for the first time,
> __cxa_thread_atexit() might be called again. I was thinking that dtors would
> get re-initialized in that case but it appears
rsmith added inline comments.
Comment at: include/clang/Basic/DiagnosticSemaKinds.td:7907
@@ -7904,1 +7906,3 @@
+def ext_opencl_ext_vector_type_rgba_selector: ExtWarn<
+ "vector component name '%0' is an OpenCL version 2.2 feature">;
} // end of sema category
As
hokein created this revision.
hokein added a reviewer: aaron.ballman.
hokein added a subscriber: cfe-commits.
Herald added a subscriber: klimek.
http://reviews.llvm.org/D21860
Files:
docs/LibASTMatchersReference.html
include/clang/ASTMatchers/ASTMatchers.h
lib/ASTMatchers/Dynamic/Registry.c
hokein marked an inline comment as done.
Comment at: clang-tidy/misc/UnusedUsingDeclsCheck.cpp:22
@@ +21,3 @@
+namespace {
+// FIXME: Move this node matcher to ASTMatcher.
+const internal::VariadicDynCastAllOfMatcher enumType;
Done in D21860.
http://reviews.llvm
hokein updated this revision to Diff 62278.
hokein added a comment.
Address review comments.
http://reviews.llvm.org/D21833
Files:
clang-tidy/misc/UnusedUsingDeclsCheck.cpp
test/clang-tidy/misc-unused-using-decls.cpp
Index: test/clang-tidy/misc-unused-using-decls.cpp
==
Author: mgottesman
Date: Wed Jun 29 15:22:44 2016
New Revision: 274157
URL: http://llvm.org/viewvc/llvm-project?rev=274157&view=rev
Log:
[cmake] Move creation of ClangTargets and installation of ClangConfig.cmake
from ./CMakeLists.txt -> ./cmake/modules/CMakeLists.txt.
This matches LLVM.
Added:
Author: mgottesman
Date: Wed Jun 29 15:22:46 2016
New Revision: 274158
URL: http://llvm.org/viewvc/llvm-project?rev=274158&view=rev
Log:
[ClangConfig] Replace paths with the same value as CLANG_INSTALL_PACKAGE_DIR
with a deref of the variable.
Modified:
cfe/trunk/cmake/modules/CMakeLists.txt
On Wed, Jun 29, 2016 at 12:51 PM, Richard Smith
wrote:
> On Wed, Jun 29, 2016 at 12:30 PM, Sean Silva
> wrote:
>
>>
>>
>> On Wed, Jun 29, 2016 at 12:09 PM, Richard Smith via cfe-commits <
>> cfe-commits@lists.llvm.org> wrote:
>>
>>> Hi all!
>>>
>>> I'd like to establish a policy for Clang's defa
On Wed, Jun 29, 2016 at 1:34 PM, Sean Silva wrote:
>
>
> On Wed, Jun 29, 2016 at 12:51 PM, Richard Smith
> wrote:
>
>> On Wed, Jun 29, 2016 at 12:30 PM, Sean Silva
>> wrote:
>>
>>>
>>>
>>> On Wed, Jun 29, 2016 at 12:09 PM, Richard Smith via cfe-commits <
>>> cfe-commits@lists.llvm.org> wrote:
>
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
LGTM, thank you so much for your patience and hard work on this change!
http://reviews.llvm.org/D18035
___
cfe-commits mailing list
cfe-commits@l
jakev updated this revision to Diff 62282.
jakev marked 5 inline comments as done.
jakev added a comment.
Address comments.
Repository:
rL LLVM
http://reviews.llvm.org/D21823
Files:
include/clang/Driver/Options.td
lib/Driver/ToolChain.cpp
lib/Driver/Tools.cpp
test/Driver/clang_f_opts
xur added a comment.
Thanks for doing this. I like the patch in general. Hers are my minor
feedbacks:
(1) It seems the original options of -fprofile-instr-generate and
-fprofile-instr-use will be kept. Is so, what is the guideline to use these two
sets of options. Also, we need some documentat
amccarth marked 2 inline comments as done.
amccarth added a comment.
http://reviews.llvm.org/D21705
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
amccarth updated this revision to Diff 62283.
amccarth added a comment.
Applied suggestion from the comments, which eliminates the need to alter the
ObjectiveC test.
http://reviews.llvm.org/D21705
Files:
lib/CodeGen/CGDebugInfo.cpp
lib/CodeGen/CGDebugInfo.h
test/CodeGenCXX/debug-info-dup
Author: rsmith
Date: Wed Jun 29 16:06:51 2016
New Revision: 274165
URL: http://llvm.org/viewvc/llvm-project?rev=274165&view=rev
Log:
Switch to an RAII object to revert tentative parsing automatically.
Modified:
cfe/trunk/include/clang/Parse/Parser.h
cfe/trunk/lib/Parse/ParseTentative.cpp
- Original Message -
> From: "Richard Smith"
> To: "Hal Finkel"
> Cc: "Clang Dev" , "cfe-commits"
>
> Sent: Wednesday, June 29, 2016 3:01:46 PM
> Subject: Re: [cfe-dev] RFC: Default language standard mode policy
> On Wed, Jun 29, 2016 at 12:55 PM, Hal Finkel via cfe-dev <
> cfe-...@lis
Author: mgottesman
Date: Wed Jun 29 16:12:38 2016
New Revision: 274168
URL: http://llvm.org/viewvc/llvm-project?rev=274168&view=rev
Log:
[ClangConfig] Copy ClangConfig.cmake to
${CMAKE_BINARY_DIR}/${CLANG_INSTALL_PACKAGE_DIR} instead of to
${CLANG_BINARY_DIR}/${CLANG_INSTALL_PACKAGE_DIR}.
This
Author: rsmith
Date: Wed Jun 29 16:12:37 2016
New Revision: 274167
URL: http://llvm.org/viewvc/llvm-project?rev=274167&view=rev
Log:
Convert another case over to RevertingTentativeParsingAction.
Modified:
cfe/trunk/lib/Parse/ParseTentative.cpp
Modified: cfe/trunk/lib/Parse/ParseTentative.cpp
Author: rsmith
Date: Wed Jun 29 16:17:59 2016
New Revision: 274169
URL: http://llvm.org/viewvc/llvm-project?rev=274169&view=rev
Log:
P0305R1: Parsing support for init-statements in 'if' and 'switch' statements.
No semantic analysis yet.
This is a pain to disambiguate correctly, because the parsin
rsmith requested changes to this revision.
rsmith added a comment.
This revision now requires changes to proceed.
The proposal has changed since the pre-meeting mailing, and now covers `switch`
as well as `if`. You also need to handle the //expression-statement// form of
//init-statement//.
As
joerg added a subscriber: joerg.
joerg added a comment.
On the topic of __cxa_thread_atexit, was it ever specified how it interacts
with things like thread cancellation?
http://reviews.llvm.org/D21803
___
cfe-commits mailing list
cfe-commits@lists.
On Wed, Jun 29, 2016 at 11:00:26AM -0400, Rafael Espíndola via cfe-commits
wrote:
> We need to benchmark this to see if it still makes a difference.
>
> My canonical test is the .s of a lto of clang.
That won't exercise it? Assembler sources are exempt from this.
Joerg
_
On Wed, Jun 29, 2016 at 12:09:37PM -0700, Richard Smith via cfe-dev wrote:
> Hi all!
>
> I'd like to establish a policy for Clang's default language standard (if
> none is specified with -std), as follows:
>
> Clang defaults to the most recent published standard for the selected
> language that
Author: mgottesman
Date: Wed Jun 29 16:59:23 2016
New Revision: 274180
URL: http://llvm.org/viewvc/llvm-project?rev=274180&view=rev
Log:
[ClangConfig] Store all of the targets exported in the variable
CLANG_EXPORTED_TARGETS.
Modified:
cfe/trunk/cmake/modules/CMakeLists.txt
cfe/trunk/cmak
Author: mgottesman
Date: Wed Jun 29 16:59:15 2016
New Revision: 274177
URL: http://llvm.org/viewvc/llvm-project?rev=274177&view=rev
Log:
[ClangConfig] Follow LLVM's example and only install Clang{Config,Target}.cmake
when LLVM_INSTALL_TOOLCHAIN_ONLY is disabled.
Modified:
cfe/trunk/cmake/mod
Author: mgottesman
Date: Wed Jun 29 16:59:19 2016
New Revision: 274178
URL: http://llvm.org/viewvc/llvm-project?rev=274178&view=rev
Log:
[ClangConfig] Instead of hard coding the ClangTargets location to
/ClangTargets.cmake, follow LLVM's example and use a pre-computed cmake
variable @CLANG_CONFI
Author: mgottesman
Date: Wed Jun 29 16:59:20 2016
New Revision: 274179
URL: http://llvm.org/viewvc/llvm-project?rev=274179&view=rev
Log:
[ClangConfig] Unset some variables after we are done using them to configure
ClangConfig.cmake.in files.
This ensures that the values do not bleed over in betw
Author: mgottesman
Date: Wed Jun 29 16:59:14 2016
New Revision: 274176
URL: http://llvm.org/viewvc/llvm-project?rev=274176&view=rev
Log:
[cmake] Instead of just copying ClangConfig.cmake, configure it using
ClangConfig.cmake.in.
This will allow for cmake to expand variables in ClangConfig.cmake
- Original Message -
> From: "Hal Finkel via cfe-dev"
> To: "Richard Smith"
> Cc: "Clang Dev" , "cfe-commits"
>
> Sent: Wednesday, June 29, 2016 4:17:18 PM
> Subject: Re: [cfe-dev] RFC: Default language standard mode policy
> - Original Message -
> > From: "Richard Smith"
>
On Wed, Jun 29, 2016 at 2:48 PM, Joerg Sonnenberger via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> On Wed, Jun 29, 2016 at 12:09:37PM -0700, Richard Smith via cfe-dev wrote:
> > Hi all!
> >
> > I'd like to establish a policy for Clang's default language standard (if
> > none is specified w
1 - 100 of 169 matches
Mail list logo
