krasimir updated this revision to Diff 105274.
krasimir added a comment.
- Address review comment
https://reviews.llvm.org/D34947
Files:
clangd/ClangdLSPServer.cpp
clangd/ClangdUnitStore.cpp
clangd/GlobalCompilationDatabase.cpp
clangd/GlobalCompilationDatabase.h
clangd/Protocol.cpp
krasimir created this revision.
Herald added a subscriber: klimek.
This patch updates the formatting of message fields of type `a{...}` to `a
{...}`
for proto messages.
https://reviews.llvm.org/D35015
Files:
lib/Format/TokenAnnotator.cpp
unittests/Format/FormatTestProto.cpp
unittests/For
gtbercea added inline comments.
Comment at: lib/Driver/ToolChains/Cuda.cpp:443
+
+// Get the compute capability from the -fopenmp-targets flag.
+// The default compute capability is sm_20 since this is a CUDA
hfinkel wrote:
> Is this first sentence accura
gtbercea updated this revision to Diff 105280.
gtbercea marked 3 inline comments as done.
gtbercea added a comment.
Address comments.
https://reviews.llvm.org/D34784
Files:
include/clang/Basic/DiagnosticDriverKinds.td
include/clang/Driver/Options.td
lib/Driver/ToolChains/Cuda.cpp
test/D
mclow.lists added a comment.
This doesn't work when running the libc++ tests against a non-system libc++.
(which is what all the libc++ developers and AND all the test bots do)
Repository:
rL LLVM
https://reviews.llvm.org/D34556
___
cfe-commits
I don't think you need the dllimport restriction.
On Wed, Jul 5, 2017 at 12:05 PM Alex Lorenz via Phabricator via cfe-commits
wrote:
> arphaman added a comment.
>
> Does this apply to all constexpr global variables? It could potentially
> fix https://bugs.llvm.org/show_bug.cgi?id=31860 .
>
>
> h
The LLVM verifier is complaining that dllimport have to be external linkage
and isn't happy with available_externally, is the verifier wrong?
2017-07-05 9:12 GMT-07:00 David Majnemer :
> I don't think you need the dllimport restriction.
>
> On Wed, Jul 5, 2017 at 12:05 PM Alex Lorenz via Phabrica
vladimir.plyashkun updated this revision to Diff 105283.
vladimir.plyashkun edited the summary of this revision.
vladimir.plyashkun added a comment.
- made code less verbose
- used assignment initialization form
- removed extra `.c_str()` call
Repository:
rL LLVM
https://reviews.llvm.org/D344
Author: marshall
Date: Wed Jul 5 09:37:19 2017
New Revision: 307171
URL: http://llvm.org/viewvc/llvm-project?rev=307171&view=rev
Log:
Fix a bug in regex_Iterator where it would report zero-length matches forever.
Reported as http://llvm.org/PR33681. Thanks to Karen Arutyunov for the report.
Mod
ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D34947
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/
On Wed, Jul 5, 2017 at 12:22 PM Mehdi AMINI wrote:
> The LLVM verifier is complaining that dllimport have to be external
> linkage and isn't happy with available_externally, is the verifier wrong?
>
IMO, yes. I imagine that it is fine with dllimport available_externally
functions already.
> 20
Author: erichkeane
Date: Wed Jul 5 09:43:45 2017
New Revision: 307172
URL: http://llvm.org/viewvc/llvm-project?rev=307172&view=rev
Log:
Address comments that escaped D3
Patch By: Jen Yu
Differential Revision:https://reviews.llvm.org/D34671
Modified:
cfe/trunk/include/clang/Basic/Diagno
This revision was automatically updated to reflect the committed changes.
Closed by commit rL307172: Address comments that escaped D3 (authored by
erichkeane).
Changed prior to commit:
https://reviews.llvm.org/D34671?vs=104966&id=105284#toc
Repository:
rL LLVM
https://reviews.llvm.org/D
ahatanak added a comment.
I'm not exactly sure why the test failed, but the patch was reverted in r306859.
Repository:
rL LLVM
https://reviews.llvm.org/D34556
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/m
Anastasia added a comment.
Btw, is there any reason to add testing specifically for half? Is there
anything specific to half in the implementation of this?
Comment at: test/CodeGenOpenCL/kernel-arg-info.cl:79
+kernel void foo6_2(global const volatile half * restrict
globalcon
Anastasia added a comment.
LGTM! Thanks!
https://reviews.llvm.org/D34980
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mclow.lists added a comment.
> I'm not exactly sure why the test failed, but the patch was reverted in
> r306859.
My bad, since the tests are failing due to clang whining about the system dylib
not supporting aligned new/delete on 10.12, I meant to post this on
https://reviews.llvm.org/D34574
malhar1995 updated this revision to Diff 105292.
malhar1995 added a comment.
Corrected one of the two test-cases added in the last-diff.
Repository:
rL LLVM
https://reviews.llvm.org/D34937
Files:
lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp
test/Analysis/retain-release-inline.m
In
mclow.lists added a comment.
This commit breaks all the libc++ aligned new/delete tests on Mac OS. Was that
deliberate?
Failing Tests (8):
libc++ ::
std/language.support/support.dynamic/new.delete/new.delete.array/delete_align_val_t_replace.pass.cpp
libc++ ::
std/language.support/support.
yawanng updated this revision to Diff 105293.
yawanng marked 8 inline comments as done.
https://reviews.llvm.org/D34913
Files:
clang-tidy/android/AndroidTidyModule.cpp
clang-tidy/android/CMakeLists.txt
clang-tidy/android/CloexecOpenCheck.cpp
clang-tidy/android/CloexecSocketCheck.cpp
cla
This revision was automatically updated to reflect the committed changes.
Closed by commit rL307175: [Sema] Don't allow -Wunguarded-availability to be
silenced with redecls (authored by epilk).
Changed prior to commit:
https://reviews.llvm.org/D33816?vs=104437&id=105296#toc
Repository:
rL LL
Author: epilk
Date: Wed Jul 5 10:08:56 2017
New Revision: 307175
URL: http://llvm.org/viewvc/llvm-project?rev=307175&view=rev
Log:
[Sema] Don't allow -Wunguarded-availability to be silenced with redecls
Differential revision: https://reviews.llvm.org/D33816
Modified:
cfe/trunk/include/clang
ilya-biryukov added a comment.
In https://reviews.llvm.org/D34936#799589, @mibintc wrote:
> Thanks so much!
You're welcome.
Repository:
rL LLVM
https://reviews.llvm.org/D34936
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://list
boris created this revision.
Extend the -fmodule-file option to support the [=] value format.
If the name is omitted, then the old semantics is preserved (the module file
is loaded whether needed or not). If the name is specified, then the mapping
is treated as just another prebuilt module search
The thunks are (as far as I know, at least) only referenced from vtables, so
there's no opportunity to perform indirect calls via the IAT for them. The
vtable would just end up referencing the linker-generated import thunk
instead.
I can play around with marking the thunks as dllimport if the func
dcoughlin accepted this revision.
dcoughlin added a comment.
This revision is now accepted and ready to land.
Sorry for the long delay! This looks good to me. Do you have commit access, or
do you need someone to commit it for you?
> Regarding " I think it would also be good to (eventually) add C
No, it was not deliberate. I'll take a look at the failing tests today.
> On Jul 5, 2017, at 10:02 AM, Marshall Clow via Phabricator via cfe-commits
> wrote:
>
> mclow.lists added a comment.
>
> This commit breaks all the libc++ aligned new/delete tests on Mac OS. Was
> that deliberate?
>
>
yaxunl marked 12 inline comments as done.
yaxunl added inline comments.
Comment at: include/clang/Basic/TargetInfo.h:969
+ /// LangAS::Default.
+ virtual unsigned getGlobalAddressSpace() const { return LangAS::Default; }
+
rjmccall wrote:
> I'm not sure this re
If the thinks are not imported, why would it make sense to export them?
Maybe devirtualization could trigger this?
On Wed, Jul 5, 2017 at 1:17 PM Shoaib Meenai wrote:
> The thunks are (as far as I know, at least) only referenced from vtables,
> so
> there's no opportunity to perform indirect ca
yaxunl updated this revision to Diff 105310.
yaxunl marked 6 inline comments as done.
yaxunl retitled this revision from "[AMDGPU] Fix address space of global
variable" to "CodeGen: Fix address space of global variable".
yaxunl edited the summary of this revision.
yaxunl added a comment.
Revised
mclow.lists added a comment.
@smeenai wrote:
> This is kinda ugly, but I can't think of a better way to do it. I'm fine with
> this, but given that it's a pretty invasive change, I'm not comfortable
> accepting. You may wanna ping @EricWF and @mclow.lists directly.
I'll be meeting with the MS
I had initially also thought that thunks wouldn't need to be exported, since
you should always have one of two scenarios:
1. The class doesn't have a key function, in which case both its vtable and
its thunks will just be emitted wherever they're needed.
2. The class has a key function, in whic
gtbercea added inline comments.
Comment at: test/Driver/openmp-offload.c:607
+
+// CHK-FOPENMP-EQ-TARGET: clang{{.*}} argument unused during compilation:
'-Xopenmp-target=powerpc64le-ibm-linux-gnu -march=pwr8'
+
hfinkel wrote:
> I don't see why you'd check that
jyknight added a comment.
Ping. I don't think this got resolved, and I really wouldn't like to see
released in this state...can you either revert this from the library, or
implement the compiler support, before the 5.0 release branch?
In https://reviews.llvm.org/D31022#716702, @jyknight wrote:
dcoughlin added a comment.
> @dcoughlin As a reviewer of both patches - could you tell us what's the
> difference between them? And how are we going to resolve this issue?
These two patches are emitting markers in the CFG for different things.
Here is how I would characterize the difference bet
STL_MSFT added a comment.
Ping?
https://reviews.llvm.org/D34536
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mehdi_amini added inline comments.
Comment at: clang/test/CodeGenCXX/cxx11-extern-constexpr.cpp:12
+// CHECK: @_ZN3Foo4BAR2E = external constant i32,
+ static const int BAR2 = 43;
+};
Note: I'm not sure if the standard allows us to assume that we can fold the
EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.
LGTM. @STL_MSFT This seems like you could commit it without review. You're a
professional STL developer after all, if you can't get this right may God have
mercy on your users.
https://revie
a.sidorin added a comment.
Oh, I see now. I was wrongly thinking that these patches do the same thing and
we're duplicating the work. Thank you very much for your explanation, Devin.
Repository:
rL LLVM
https://reviews.llvm.org/D16403
___
cfe-co
STL_MSFT added a comment.
Yeah, I just wanted to check that you didn't mean to actually use these
typedefs for something. Thanks!
https://reviews.llvm.org/D34536
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/
STL_MSFT added a comment.
Ping for this one - there's no functional change here, but since I don't have
any glibc coverage, I wanted to get a review.
https://reviews.llvm.org/D34535
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lis
yaxunl added a comment.
Sorry for the delay.
The spec says:
The data type names described in the table below are reserved and cannot be
used by applications as user-defined type names.
However, it does not forbid using the reserved types as builtin types if the
compiler supports it.
I would
yaxunl accepted this revision.
yaxunl added a comment.
LGTM. Thanks!
https://reviews.llvm.org/D33109
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mehdi_amini accepted this revision.
mehdi_amini added a comment.
Make sure your commit message correctly reference the LLVM change.
Comment at: test/CodeGen/ms-barriers-intrinsics.c:39
#endif
-
Don't remove: file should end with a new line I believe.
https:/
FWIW, doesn't this just need a may_alias attribute on the definition?
On Wed, Jul 5, 2017 at 2:39 PM James Y Knight via Phabricator via
cfe-commits wrote:
> jyknight added a comment.
>
> Ping. I don't think this got resolved, and I really wouldn't like to see
> released in this state...can you ei
gtbercea updated this revision to Diff 105320.
gtbercea edited the summary of this revision.
gtbercea added a comment.
Rebase on latest master.
https://reviews.llvm.org/D29658
Files:
lib/Driver/Driver.cpp
Index: lib/Driver/Driver.cpp
=
Author: modocache
Date: Wed Jul 5 12:55:51 2017
New Revision: 307193
URL: http://llvm.org/viewvc/llvm-project?rev=307193&view=rev
Log:
Add docs for -foptimization-record-file=
Summary: The Clang option was previously not included in the User's Manual.
Reviewers: anemet, davidxl, hfinkel
Review
gtbercea added inline comments.
Comment at: lib/Driver/Driver.cpp:564
+ auto &CudaTC =
+ ToolChains[TT.str() + "/" + HostTC->getTriple().normalize()];
+ if (!CudaTC)
Hahnfeld wrote:
> The code above uses `HostTriple.str(
modocache added a comment.
Thanks for the review, Hal! :)
https://reviews.llvm.org/D34928
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
gtbercea added a comment.
In https://reviews.llvm.org/D29658#732520, @ABataev wrote:
> Tests?
Can't write any meaningful tests. This will be tested by all future patches
that perform offloading using OpenMP.
https://reviews.llvm.org/D29658
___
c
mibintc added a comment.
In https://reviews.llvm.org/D34158#797968, @Hahnfeld wrote:
> In https://reviews.llvm.org/D34158#797170, @mibintc wrote:
>
> > The other test that fails is my own new test! It fails because I don't know
> > how to set it up so the test thinks it has a gcc toolchain with
Author: dgregor
Date: Wed Jul 5 13:20:14 2017
New Revision: 307196
URL: http://llvm.org/viewvc/llvm-project?rev=307196&view=rev
Log:
Customize the SFINAE diagnostics for enable_if to provide the failed condition.
When enable_if disables a particular overload resolution candidate,
rummage through
Author: dgregor
Date: Wed Jul 5 13:20:15 2017
New Revision: 307197
URL: http://llvm.org/viewvc/llvm-project?rev=307197&view=rev
Log:
Cope with Range-v3's CONCEPT_REQUIRES idiom
Modified:
cfe/trunk/lib/Sema/SemaTemplate.cpp
cfe/trunk/test/SemaTemplate/overload-candidates.cpp
Modified: cf
modocache planned changes to this revision.
modocache added a comment.
Oh, nice catch @jordan_rose, you're absolutely right. I just tried updating my
Git submodule reference for Clang and rebuilding my project, but the commit
hash shown for Clang didn't change accordingly.
I'll need to find a f
pcc added a comment.
FYI, I don't think `HEAD` on its own will work because it is not necessarily
updated on every commit, see discussion on https://reviews.llvm.org/D31985.
https://reviews.llvm.org/D34955
___
cfe-commits mailing list
cfe-commits@l
On Wed, Jul 5, 2017 at 4:20 PM, Douglas Gregor via cfe-commits
wrote:
> Author: dgregor
> Date: Wed Jul 5 13:20:15 2017
> New Revision: 307197
>
> URL: http://llvm.org/viewvc/llvm-project?rev=307197&view=rev
> Log:
> Cope with Range-v3's CONCEPT_REQUIRES idiom
>
> Modified:
> cfe/trunk/lib/Se
jordan_rose added a comment.
I'm not sure why we would care if a commit is made on a branch. The important
information here is what commit is checked out, and that's what HEAD refers to.
https://reviews.llvm.org/D34955
___
cfe-commits mailing list
mibintc updated this revision to Diff 105329.
mibintc added a comment.
I responded to the review from Jonas Hahnfeld: I moved the entire change into
ToolChains/Linux and removed the modifications to Gnu.h and Gnu.cpp; I modified
the new tests to use -### with FileCheck, and added a tree in Input
v.g.vassilev accepted this revision.
v.g.vassilev added a comment.
This revision is now accepted and ready to land.
LGTM.
https://reviews.llvm.org/D34927
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/l
> On Jul 5, 2017, at 1:33 PM, Aaron Ballman wrote:
>
> On Wed, Jul 5, 2017 at 4:20 PM, Douglas Gregor via cfe-commits
> mailto:cfe-commits@lists.llvm.org>> wrote:
>> Author: dgregor
>> Date: Wed Jul 5 13:20:15 2017
>> New Revision: 307197
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=3071
Author: mzolotukhin
Date: Wed Jul 5 14:06:11 2017
New Revision: 307201
URL: http://llvm.org/viewvc/llvm-project?rev=307201&view=rev
Log:
Fix one more reference to lit.util.capture()
The capture method was removed in r306643.
Modified:
cfe/trunk/utils/perf-training/lit.cfg
cfe/trunk/util
mibintc added a comment.
Jonas asked about adding a new test to ensure that "-include stdc-predef.h"
does not get added if the file doesn't exist.
I added a reply to that but I can't see where it went. So I'm writing the reply
again.
The current version of the patch doesn't check for the exist
Author: dgregor
Date: Wed Jul 5 14:12:37 2017
New Revision: 307202
URL: http://llvm.org/viewvc/llvm-project?rev=307202&view=rev
Log:
Fix test case in pre-C++11 mode; address Aaron Ballman's code review.
Modified:
cfe/trunk/lib/Sema/SemaTemplate.cpp
cfe/trunk/test/SemaTemplate/overload-ca
On 5 July 2017 at 14:05, Douglas Gregor via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
>
> On Jul 5, 2017, at 1:33 PM, Aaron Ballman wrote:
>
> On Wed, Jul 5, 2017 at 4:20 PM, Douglas Gregor via cfe-commits
> wrote:
>
> Author: dgregor
> Date: Wed Jul 5 13:20:15 2017
> New Revision: 30719
On Wed, Jul 5, 2017 at 5:05 PM, Douglas Gregor wrote:
>
> On Jul 5, 2017, at 1:33 PM, Aaron Ballman wrote:
>
> On Wed, Jul 5, 2017 at 4:20 PM, Douglas Gregor via cfe-commits
> wrote:
>
> Author: dgregor
> Date: Wed Jul 5 13:20:15 2017
> New Revision: 307197
>
> URL: http://llvm.org/viewvc/llvm-
I actually didn't know that attribute existed. It does look like the right
thing to use here. However, unfortunately, it appears it doesn't work for
enum class in clang [[1]]. It *does* seem to work in GCC.
Unfortunately, it seems that GCC has actually hardcoded the type name
"std::byte" (in gcc/c
Hello,
the attached patch introduces a -mtls-gs command line option for clang
for the x86_64 target.
This option will force the usage of the GS instead of the FS register
for TLS references and
stack canary references.
This is particularly useful for low-level kernel-side development.
Best re
efriedma added a comment.
Needs a testcase (to check the emitted IR).
Comment at: lib/CodeGen/TargetInfo.cpp:4769
DarwinPCS
};
Can we extend ABIKind rather than adding a separate boolean?
https://reviews.llvm.org/D35008
__
I missed the second part.
Am 05.07.2017 um 23:38 schrieb svk:
Hello,
the attached patch introduces a -mtls-gs command line option for clang
for the x86_64 target.
This option will force the usage of the GS instead of the FS register
for TLS references and
stack canary references.
This is pa
On 5 July 2017 at 14:29, James Y Knight wrote:
> I actually didn't know that attribute existed. It does look like the right
> thing to use here. However, unfortunately, it appears it doesn't work for
> enum class in clang [[1]]. It *does* seem to work in GCC.
>
> Unfortunately, it seems that GCC
On 5 July 2017 at 14:09, Melanie Blower via Phabricator <
revi...@reviews.llvm.org> wrote:
> mibintc added a comment.
>
> Jonas asked about adding a new test to ensure that "-include
> stdc-predef.h" does not get added if the file doesn't exist.
>
> I added a reply to that but I can't see where it
ddcc added a comment.
ping
https://reviews.llvm.org/D28953
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: ericwf
Date: Wed Jul 5 15:40:58 2017
New Revision: 307218
URL: http://llvm.org/viewvc/llvm-project?rev=307218&view=rev
Log:
Suppress -Waligned-allocation-unavailable warning in libc++ test suite
Modified:
libcxx/trunk/utils/libcxx/test/config.py
Modified: libcxx/trunk/utils/libcxx/t
yawanng added a comment.
PING
Repository:
rL LLVM
https://reviews.llvm.org/D34114
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ahatanak added inline comments.
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:2376
+const VarDecl *InitDecl;
+const Expr *InitExpr = D->getAnyInitializer(InitDecl);
+if (InitExpr) {
Does getAnyInitializer ever return a null pointer here when D is a
mehdi_amini added inline comments.
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:2376
+const VarDecl *InitDecl;
+const Expr *InitExpr = D->getAnyInitializer(InitDecl);
+if (InitExpr) {
ahatanak wrote:
> Does getAnyInitializer ever return a null poin
rsmith added a comment.
We've had problems in the past with speculative emission of values like this
resulting in us producing invalid symbol references. (Two cases I remember: an
`available_externally` symbol references a symbol that should be emitted as
`linkonce_odr` but which is not emitted
gtbercea updated this revision to Diff 105354.
gtbercea added a comment.
Address comments.
https://reviews.llvm.org/D34784
Files:
include/clang/Basic/DiagnosticDriverKinds.td
include/clang/Driver/Options.td
lib/Driver/Driver.cpp
lib/Driver/ToolChains/Clang.cpp
lib/Driver/ToolChains/Co
gtbercea updated this revision to Diff 105355.
gtbercea added a comment.
Address Comments.
https://reviews.llvm.org/D34784
Files:
include/clang/Basic/DiagnosticDriverKinds.td
include/clang/Driver/Options.td
lib/Driver/ToolChains/Cuda.cpp
test/Driver/openmp-offload.c
Index: test/Driver/
mehdi_amini added inline comments.
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:2374
+ llvm::Constant *Init = nullptr;
+ if (D && D->isConstexpr() && !D->isInline() && !D->hasAttr())
{
+const VarDecl *InitDecl;
rsmith wrote:
> Applying this for only `co
rsmith added a comment.
Some concrete suggestions throughout the patch, but I think we should take a
step back and reconsider this warning approach: it seems bizarre for us to warn
on any packed struct that happens to contain a `char`. It would make sense to
warn if an `__attribute__((packed))`
gtbercea added inline comments.
Comment at: lib/Driver/ToolChains/Cuda.cpp:478
+auto MArchList = DAL->getAllArgValues(options::OPT_march_EQ);
+assert(MArchList.size() < 2 && "At most one GPU arch allowed.");
+if (MArchList.empty())
hfinkel wrote:
> Ca
rsmith added inline comments.
Comment at: clang/test/CodeGenCXX/cxx11-extern-constexpr.cpp:15
+// CHECK: @_ZL4BAR3 = available_externally constant i32 44,
+static constexpr int BAR3 = 44;
+
mehdi_amini wrote:
> rsmith wrote:
> > mehdi_amini wrote:
> > > Looks lik
jroelofs created this revision.
Herald added a subscriber: mgorny.
Mostly cargo-culted from libcxxabi, since the unwinder was forked from there in
the first place.
Might still be some cruft that's only applicable to libcxxabi in here, so let
me know if you spot anything like that. I killed some
rsmith added a comment.
We need to be completely clear on the differences and interactions between
`_Float16` and `__fp16`; you should include a documentation update that
describes this as part of this change or as a companion change. In particular,
an explanation of why we need two different t
Author: ericwf
Date: Wed Jul 5 17:29:09 2017
New Revision: 307230
URL: http://llvm.org/viewvc/llvm-project?rev=307230&view=rev
Log:
Fix incomplete type test on OS X; workaround weird DYLD_LIBRARY_PATH behavior
Modified:
libcxxabi/trunk/test/incomplete_type.sh.cpp
Modified: libcxxabi/trunk/t
Author: rsmith
Date: Wed Jul 5 17:29:13 2017
New Revision: 307231
URL: http://llvm.org/viewvc/llvm-project?rev=307231&view=rev
Log:
[cxx_status] Update link to Modules TS to latest working draft. Fix Coroutines
TS flag to work if copy-pasted.
Modified:
cfe/trunk/www/cxx_status.html
Modifie
Author: rsmith
Date: Wed Jul 5 17:30:00 2017
New Revision: 307232
URL: http://llvm.org/viewvc/llvm-project?rev=307232&view=rev
Log:
[modules ts] Do not emit strong function definitions from the module interface
unit in every user.
Added:
cfe/trunk/test/CXX/modules-ts/basic/basic.def.odr/
EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.
This LGTM. I'm assuming it works but we can iron out the kinks once it lands.
We should probably add builders that run the tests as well.
Comment at: test/lit.cfg:46
+li
EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.
> Creating a function pointer with proper parameters pointing to std::next() or
> std::prev() should work.
Actually, according to the standard taking the address of a STL function is UB,
and
ahatanak added a subscriber: EricWF.
ahatanak added a comment.
Does r307218 unbreak the tests?
Repository:
rL LLVM
https://reviews.llvm.org/D34574
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listi
phosek accepted this revision.
phosek added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D34489
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
dlj added a comment.
In https://reviews.llvm.org/D34853#798699, @andrewng wrote:
> Hi,
>
> I believe that this "build mode" is intended for the Visual Studio MSVC
> build. This build is special in that it can produce builds for multiple
> configurations, e.g. Debug, Release & RelWithDebInfo, wi
alexshap created this revision.
Herald added a subscriber: xazax.hun.
This is a follow up for one of the previous diffs
https://reviews.llvm.org/D32328.
getTypeSize and with getIntWidth are not equivalent for bool
(see https://clang.llvm.org/doxygen/ASTContext_8cpp_source.html#l08444),
this caus
jyknight added inline comments.
Comment at: lib/Driver/ToolChains/Linux.cpp:755
+!Version.isOlderThan(4, 8, 0)) {
+ // For gcc >= 4.8.x, clang will preinclude
+ // -ffreestanding suppresses this behavior.
I don't see why it makes any sense to c
mgrang added a comment.
Ping for reviews please.
https://reviews.llvm.org/D34860
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.
Sounds plausible to me
https://reviews.llvm.org/D34860
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/
Hahnfeld added a comment.
In https://reviews.llvm.org/D34888#799576, @gtbercea wrote:
> Does this also include the fixes in the following revision?
>
> https://reviews.llvm.org/D29905
Sorry, I wasn't aware of this revision and thought that it had long been
committed. I just verified that the b
Hahnfeld added inline comments.
Comment at: lib/Driver/Driver.cpp:564
+ auto &CudaTC =
+ ToolChains[TT.str() + "/" + HostTC->getTriple().normalize()];
+ if (!CudaTC)
gtbercea wrote:
> Hahnfeld wrote:
> > The code above u
echuraev updated this revision to Diff 105375.
echuraev marked an inline comment as done.
https://reviews.llvm.org/D35000
Files:
test/CodeGenOpenCL/kernel-arg-info.cl
Index: test/CodeGenOpenCL/kernel-arg-info.cl
===
--- test/Code
101 - 200 of 203 matches
Mail list logo