mehdi_amini added a comment.
In https://reviews.llvm.org/D28404#638299, @probinson wrote:
> In https://reviews.llvm.org/D28404#638221, @mehdi_amini wrote:
>
> > In https://reviews.llvm.org/D28404#638217, @probinson wrote:
> >
> > > The patch as-is obviously has a massive testing cost, and it's ea
mehdi_amini updated this revision to Diff 83433.
mehdi_amini added a comment.
Herald added a subscriber: jholewinski.
Fix minsize issue (conditional was reversed)
https://reviews.llvm.org/D28404
Files:
clang/include/clang/Driver/CC1Options.td
clang/include/clang/Frontend/CodeGenOptions.def
On Fri, Jan 6, 2017 at 12:16 PM Eric Fiselier via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: ericwf
> Date: Fri Jan 6 14:05:40 2017
> New Revision: 291275
>
> URL: http://llvm.org/viewvc/llvm-project?rev=291275&view=rev
> Log:
> [libc++] Cleanup and document <__threading_support>
Author: rsmith
Date: Fri Jan 6 16:52:53 2017
New Revision: 291295
URL: http://llvm.org/viewvc/llvm-project?rev=291295&view=rev
Log:
Revisit PR10177: don't instantiate a variable if it's only referenced in a
dependent context and can't be used in a constant expression.
Per C++ [temp.inst]p2, "the
mehdi_amini updated this revision to Diff 83441.
mehdi_amini added a comment.
Herald added subscribers: dschuff, jfb.
Fix one more conflicts with always_inline, and change some test check lines
https://reviews.llvm.org/D28404
Files:
clang/include/clang/Driver/CC1Options.td
clang/include/cla
EricWF added a comment.
I'm seeing the following build errors on Windows:
C:\Users\Eric\workspace\libcxx\include\__threading_support(521,3): warning:
cannot delete expression with pointer-to-'void' type 'void *'
[-Wdelete-incomplete]
delete __data;
^ ~~
C:\Users\Eric\works
Hi David,
Thanks for the information. Looking into it now.
/Eric
On Fri, Jan 6, 2017 at 4:02 PM, David Blaikie wrote:
>
>
> On Fri, Jan 6, 2017 at 12:16 PM Eric Fiselier via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
>
>> Author: ericwf
>> Date: Fri Jan 6 14:05:40 2017
>> New Revision
chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D28385
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo
Should be fixed in r291298. Sorry for the breakage.
/Eric
On Fri, Jan 6, 2017 at 4:18 PM, Eric Fiselier wrote:
> Hi David,
>
> Thanks for the information. Looking into it now.
>
> /Eric
>
> On Fri, Jan 6, 2017 at 4:02 PM, David Blaikie wrote:
>
>>
>>
>> On Fri, Jan 6, 2017 at 12:16 PM Eric Fis
Author: ericwf
Date: Fri Jan 6 17:15:16 2017
New Revision: 291298
URL: http://llvm.org/viewvc/llvm-project?rev=291298&view=rev
Log:
Fix breakage caused when _LIBCPP_HAS_THREAD_API_PTHREAD is manually defined
Modified:
libcxx/trunk/include/__config
Modified: libcxx/trunk/include/__config
URL
Author: mehdi_amini
Date: Fri Jan 6 17:18:09 2017
New Revision: 291300
URL: http://llvm.org/viewvc/llvm-project?rev=291300&view=rev
Log:
Add a cc1 option to force disabling lifetime-markers emission from clang
Summary: This intended as a debugging/development flag only.
Differential Revision: h
This revision was automatically updated to reflect the committed changes.
Closed by commit rL291300: Add a cc1 option to force disabling lifetime-markers
emission from clang (authored by mehdi_amini).
Changed prior to commit:
https://reviews.llvm.org/D28385?vs=83321&id=83443#toc
Repository:
jgravelle-google added a comment.
In https://reviews.llvm.org/D28381#637915, @sunfish wrote:
> Would it be difficult to enable atomic.c in Emscripten's compiler-rt build,
> to define these libcalls (assuming that the problem is just that they're not
> currently defined)?
Tried it and it seems
probinson added a comment.
In https://reviews.llvm.org/D28404#638350, @mehdi_amini wrote:
> In https://reviews.llvm.org/D28404#638299, @probinson wrote:
>
> > In https://reviews.llvm.org/D28404#638221, @mehdi_amini wrote:
> >
> > > In https://reviews.llvm.org/D28404#638217, @probinson wrote:
> >
chandlerc added inline comments.
Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:760-762
Fn->removeFnAttr(llvm::Attribute::NoInline);
+ Fn->removeFnAttr(llvm::Attribute::OptimizeNone);
Fn->addFnAttr(llvm::Attribute::AlwaysInline);
At point where we are i
dyung created this revision.
dyung added a reviewer: ABataev.
dyung added a subscriber: cfe-commits.
This test as originally written fails when run with a compiler that defaults to
a different C++ standard. This patch fixes up the test and adds testing it
explicitly using c++98 and c++11.
http
Author: tejohnson
Date: Fri Jan 6 17:37:33 2017
New Revision: 291303
URL: http://llvm.org/viewvc/llvm-project?rev=291303&view=rev
Log:
[ThinLTO] Optionally ignore empty index file
Summary:
In order to simplify distributed build system integration, where actions
may be scheduled before the Thin L
This revision was automatically updated to reflect the committed changes.
Closed by commit rL291303: [ThinLTO] Optionally ignore empty index file
(authored by tejohnson).
Changed prior to commit:
https://reviews.llvm.org/D28362?vs=83405&id=83450#toc
Repository:
rL LLVM
https://reviews.llvm.
george.burgess.iv updated this revision to Diff 83412.
george.burgess.iv marked 2 inline comments as done.
george.burgess.iv added a comment.
Addressed feedback. Thanks!
https://reviews.llvm.org/D27424
Files:
include/clang/AST/Expr.h
include/clang/Basic/Attr.td
include/clang/Basic/AttrDoc
amaiorano created this revision.
amaiorano added reviewers: malcolm.parsons, ioeric.
amaiorano added a subscriber: cfe-commits.
Certain tests expect their input files to have LF line endings, while others
expect CRLF. On Windows, when using Git with core.autocrlf=true, these tests
fail because a
george.burgess.iv added a comment.
> We do:
> http://llvm.org/docs/CodingStandards.html#use-auto-type-deduction-to-make-code-more-readable
Awesome. Thanks!
> Btw, there's a question in there about late parsing that Phab won't let me
> delete, feel free to ignore it. ;-)
OK. The answer is "yes
Thanks!
On Fri, Jan 6, 2017 at 3:26 PM Eric Fiselier wrote:
> Should be fixed in r291298. Sorry for the breakage.
>
> /Eric
>
> On Fri, Jan 6, 2017 at 4:18 PM, Eric Fiselier wrote:
>
> Hi David,
>
> Thanks for the information. Looking into it now.
>
> /Eric
>
> On Fri, Jan 6, 2017 at 4:02 PM, D
compnerd set the repository for this revision to rL LLVM.
compnerd updated this revision to Diff 83453.
Repository:
rL LLVM
https://reviews.llvm.org/D28220
Files:
include/__config
include/__threading_support
include/thread
Index: include/thread
==
probinson added inline comments.
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:962
+ ShouldAddOptNone &= !D->hasAttr();
+ ShouldAddOptNone &= !D->hasAttr();
+ ShouldAddOptNone &= !F->hasFnAttribute(llvm::Attribute::AlwaysInline);
chandlerc wrote:
> why is op
Author: ericwf
Date: Fri Jan 6 17:59:52 2017
New Revision: 291306
URL: http://llvm.org/viewvc/llvm-project?rev=291306&view=rev
Log:
Configure default ABI library as NONE on Windows
Modified:
libcxx/trunk/CMakeLists.txt
Modified: libcxx/trunk/CMakeLists.txt
URL:
http://llvm.org/viewvc/llvm-
Author: ericwf
Date: Fri Jan 6 18:01:55 2017
New Revision: 291309
URL: http://llvm.org/viewvc/llvm-project?rev=291309&view=rev
Log:
Fix linking of DLL's on Windows
On Windows the runtime search path for DLL's is the same as PATH.
This patch changes the test suite to add the libc++ build director
Author: tejohnson
Date: Fri Jan 6 18:09:42 2017
New Revision: 291310
URL: http://llvm.org/viewvc/llvm-project?rev=291310&view=rev
Log:
[ThinLTO] Specify target triple in new test
This should fix bot failures in this test.
Modified:
cfe/trunk/test/CodeGen/thinlto_backend.ll
Modified: cfe/tr
Author: ericwf
Date: Fri Jan 6 18:10:42 2017
New Revision: 291311
URL: http://llvm.org/viewvc/llvm-project?rev=291311&view=rev
Log:
Get all tuple tests passing on Windows
Modified:
libcxx/trunk/test/libcxx/test/config.py
libcxx/trunk/test/support/disable_missing_braces_warning.h
Modifie
mehdi_amini updated this revision to Diff 83459.
mehdi_amini added a comment.
Address comments: reorganize the way ShouldAddOptNone is handled, hopefully
make it more easy to track.
Also after talking with Chandler on IRC, the source attribute "cold" does
not add the LLVM IR attribute "optsize"
mehdi_amini marked 6 inline comments as done.
mehdi_amini added inline comments.
Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:760-762
Fn->removeFnAttr(llvm::Attribute::NoInline);
+ Fn->removeFnAttr(llvm::Attribute::OptimizeNone);
Fn->addFnAttr(llvm::Attribute::AlwaysI
EricWF added a comment.
Just tried the updated patch and I still get one build error:
C:\Users\Eric\workspace\libcxx\src\thread.cpp(135,16): error: use of
undeclared identifier 'nanosleep'
while (nanosleep(&ts, &ts) == -1 && errno == EINTR)
Repository:
rL LLVM
https://reviews.l
EricWF added inline comments.
Comment at: include/__threading_support:29
+#include
+#define WIN32_LEAN_AND_MEAN
+#include
I think we agreed that we cannot use this macro.
Comment at: include/__threading_support:426
+// Condition Variable
+_LI
majnemer added inline comments.
Comment at: include/__threading_support:29-30
+#include
+#define WIN32_LEAN_AND_MEAN
+#include
+#include
EricWF wrote:
> I think we agreed that we cannot use this macro.
Can we do as Reid suggests and not expose users to windows
probinson added inline comments.
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:896
+ !CodeGenOpts.DisableO0ImplyOptNone && CodeGenOpts.OptimizationLevel == 0;
+ // We can't add optnone in the following cases, it won't pass the verifier
+ ShouldAddOptNone &= !D->hasAttr()
EricWF added inline comments.
Comment at: include/__threading_support:580
+int __libcpp_tls_create(__libcpp_tls_key* __key,
+void(_LIBCPP_TLS_DESTRUCTOR_CC* __at_exit)(void*))
+{
The `_LIBCPP_TLS_DESTRUCTOR_CC` needs to appear an the initi
mehdi_amini marked 2 inline comments as done.
mehdi_amini added inline comments.
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:900
+ ShouldAddOptNone &= !D->hasAttr();
+ if (ShouldAddOptNone) {
+B.addAttribute(llvm::Attribute::OptimizeNone);
probinson wro
EricWF added inline comments.
Comment at: include/__threading_support:30
+#define WIN32_LEAN_AND_MEAN
+#include
+#include
> Can we do as Reid suggests and not expose users to windows.h?
I was about to ask the same question. These includes are dragging in the
Author: rsmith
Date: Fri Jan 6 18:48:55 2017
New Revision: 291318
URL: http://llvm.org/viewvc/llvm-project?rev=291318&view=rev
Log:
PR23135: Don't instantiate constexpr functions referenced in unevaluated
operands where possible.
This implements something like the current direction of DR1581: w
Author: rsmith
Date: Fri Jan 6 18:52:10 2017
New Revision: 291319
URL: http://llvm.org/viewvc/llvm-project?rev=291319&view=rev
Log:
PR20090: Add (passing) test from this bug; it's been fixed for a while.
Modified:
cfe/trunk/test/SemaTemplate/constexpr-instantiate.cpp
Modified: cfe/trunk/tes
mehdi_amini updated this revision to Diff 83468.
mehdi_amini added a comment.
Address Paul's comment (remove useless block and add period to end comment)
https://reviews.llvm.org/D28404
Files:
clang/include/clang/Driver/CC1Options.td
clang/include/clang/Frontend/CodeGenOptions.def
clang/l
Hi,
This is a (tested!) patch to add checks that both pre-increment and
pre-decrement don’t overflow, for both char and short integer types (since
they’re both narrower than int), as per the problem fixed in PR9350.
Thanks, Andrew R
From: Friedman, Eli [mailto:efrie...@codeaurora.org]
compnerd removed rL LLVM as the repository for this revision.
compnerd updated this revision to Diff 83469.
compnerd marked 2 inline comments as done.
compnerd added a comment.
remove `WIN32_LEAN_AND_MEAN`, fix decoration, remove inline decorations
https://reviews.llvm.org/D28220
Files:
inclu
tigerleapgorge created this revision.
tigerleapgorge added reviewers: rjmccall, rsmith.
tigerleapgorge added a subscriber: cfe-commits.
Hi everyone,
I am Charles Li at Sony Playstation.
I am refactoring my existing C++11 compatibility patches to make them easier to
review.
This patch is a subset
probinson added inline comments.
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:910-912
// OptimizeNone wins over OptimizeForSize and MinSize.
F->removeFnAttr(llvm::Attribute::OptimizeForSize);
F->removeFnAttr(llvm::Attribute::MinSize);
mehdi_amin
EricWF created this revision.
EricWF added reviewers: compnerd, smeenai, mclow.lists, majnemer, rnk, rsmith.
EricWF added a subscriber: cfe-commits.
On Windows the identifier `__deallocate` is defined as a macro by one of the
Windows system headers. Previously libc++ worked around this by `#undef
Author: stl_msft
Date: Fri Jan 6 19:12:15 2017
New Revision: 291322
URL: http://llvm.org/viewvc/llvm-project?rev=291322&view=rev
Log:
[libcxx] [test] Strip trailing whitespace. NFC, no code review.
Modified:
libcxx/trunk/test/std/containers/sequences/array/iterators.pass.cpp
libcxx/trunk
smeenai added inline comments.
Comment at: include/__threading_support:30
+#define WIN32_LEAN_AND_MEAN
+#include
+#include
EricWF wrote:
> > Can we do as Reid suggests and not expose users to windows.h?
>
> I was about to ask the same question. These includes
EricWF added inline comments.
Comment at: include/__threading_support:30
+#define WIN32_LEAN_AND_MEAN
+#include
+#include
smeenai wrote:
> EricWF wrote:
> > > Can we do as Reid suggests and not expose users to windows.h?
> >
> > I was about to ask the same que
smeenai added a comment.
I'm guessing always defining `_LIBCPP_DISABLE_MACRO_CONFLICT_WARNINGS` on
Windows isn't considered an acceptable workaround either?
https://reviews.llvm.org/D28426
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
htt
EricWF added inline comments.
Comment at: include/__threading_support:30
+#define WIN32_LEAN_AND_MEAN
+#include
+#include
EricWF wrote:
> smeenai wrote:
> > EricWF wrote:
> > > > Can we do as Reid suggests and not expose users to windows.h?
> > >
> > > I was a
smeenai added inline comments.
Comment at: include/__threading_support:30
+#define WIN32_LEAN_AND_MEAN
+#include
+#include
EricWF wrote:
> EricWF wrote:
> > smeenai wrote:
> > > EricWF wrote:
> > > > > Can we do as Reid suggests and not expose users to windows.
hamzasood updated this revision to Diff 83473.
hamzasood added a comment.
- Modify the environment in some cases in buildLinker to keep link.exe happy.
Thanks @rnk for taking the time to look at this. As suggested:
- Replaced the confusing unique_ptr usage with a struct.
- Renamed llvmArchToSubD
EricWF added a comment.
In https://reviews.llvm.org/D28426#638596, @smeenai wrote:
> I'm guessing always defining `_LIBCPP_DISABLE_MACRO_CONFLICT_WARNINGS` on
> Windows isn't considered an acceptable workaround either?
Not really, since that still breaks users of the `__deallocate` since we
`
dlj created this revision.
dlj added a reviewer: rsmith.
dlj added subscribers: cfe-commits, EricWF.
Per https://wg21.link/CWG1677, the C++11 standard did not clarify that constant
initialization of an object allowed constexpr brace-or-equal initialization of
subobjects:
struct foo_t { union {
EricWF added inline comments.
Comment at: include/__threading_support:30
+#define WIN32_LEAN_AND_MEAN
+#include
+#include
smeenai wrote:
> EricWF wrote:
> > EricWF wrote:
> > > smeenai wrote:
> > > > EricWF wrote:
> > > > > > Can we do as Reid suggests and not
EricWF added inline comments.
Comment at: include/__threading_support:30
+#define WIN32_LEAN_AND_MEAN
+#include
+#include
EricWF wrote:
> smeenai wrote:
> > EricWF wrote:
> > > EricWF wrote:
> > > > smeenai wrote:
> > > > > EricWF wrote:
> > > > > > > Can we do
mehdi_amini added inline comments.
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:910-912
// OptimizeNone wins over OptimizeForSize and MinSize.
F->removeFnAttr(llvm::Attribute::OptimizeForSize);
F->removeFnAttr(llvm::Attribute::MinSize);
probinso
mehdi_amini updated this revision to Diff 83480.
mehdi_amini added a comment.
Forgot to update test/CodeGen/attr-naked.c
https://reviews.llvm.org/D28404
Files:
clang/include/clang/Driver/CC1Options.td
clang/include/clang/Frontend/CodeGenOptions.def
clang/lib/CodeGen/CGOpenMPRuntime.cpp
echristo added a reviewer: scanon.
echristo added a comment.
Adding Steve in an attempt to get him to review :)
https://reviews.llvm.org/D27872
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cf
On Fri, Jan 6, 2017 at 5:56 AM Hal Finkel wrote:
>
> On 01/05/2017 08:30 PM, Eric Christopher via cfe-commits wrote:
>
> Ok, thanks. I agree that it's a problem. I'm definitely open for testing
> ideas here. There are a few other things in the
> TargetOptions/MCTargetOptions area that are already
chandlerc added inline comments.
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:910-912
// OptimizeNone wins over OptimizeForSize and MinSize.
F->removeFnAttr(llvm::Attribute::OptimizeForSize);
F->removeFnAttr(llvm::Attribute::MinSize);
mehdi_amin
Author: ericwf
Date: Fri Jan 6 20:43:58 2017
New Revision: 291329
URL: http://llvm.org/viewvc/llvm-project?rev=291329&view=rev
Log:
Add _LIBCPP_ABI_[ITANIUM|MICROSOFT] macros.
This patch adds a libc++ configuration macro for the ABI we
are targeting, either Itanium or Microsoft. For now we confi
Author: smeenai
Date: Fri Jan 6 20:45:35 2017
New Revision: 291330
URL: http://llvm.org/viewvc/llvm-project?rev=291330&view=rev
Log:
[libc++] Correct macro name in documenation
The macro is named `_LIBCPP_TEMPLATE_VIS`, not `_LIBCPP_TEMPLATE_ONLY`.
No functional change.
Modified:
libcxx/tru
Author: compnerd
Date: Fri Jan 6 20:48:30 2017
New Revision: 291331
URL: http://llvm.org/viewvc/llvm-project?rev=291331&view=rev
Log:
thread: implement sleep_for on Windows
Windows does not provide an implementation of `nanosleep`. Round up the
time duration to the nearest ms and use `Sleep`.
compnerd accepted this revision.
compnerd added a comment.
This revision is now accepted and ready to land.
This is a really nice cleanup!
https://reviews.llvm.org/D28426
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/
EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.
LGTM. I just successfully built this on Windows.
https://reviews.llvm.org/D28220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://l
Author: ericwf
Date: Fri Jan 6 21:01:24 2017
New Revision: 291332
URL: http://llvm.org/viewvc/llvm-project?rev=291332&view=rev
Log:
[libc++] Tolerate presence of __deallocate macro
Summary:
On Windows the identifier `__deallocate` is defined as a macro by one of the
Windows system headers. Prev
compnerd updated this revision to Diff 83489.
compnerd added a comment.
Rebase for r291329.
Repository:
rL LLVM
https://reviews.llvm.org/D28212
Files:
include/typeinfo
src/typeinfo.cpp
Index: src/typeinfo.cpp
===
--- src/ty
Author: compnerd
Date: Fri Jan 6 21:07:45 2017
New Revision: 291333
URL: http://llvm.org/viewvc/llvm-project?rev=291333&view=rev
Log:
provide Win32 native threading
Add an implementation for the Win32 threading model as a backing API for
the internal c++ threading interfaces. This uses the Fls*
compnerd closed this revision.
compnerd added a comment.
SVN r291333
Comment at: include/__threading_support:29-30
+#include
+#define WIN32_LEAN_AND_MEAN
+#include
+#include
majnemer wrote:
> EricWF wrote:
> > I think we agreed that we cannot use this macro.
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
Looks good to me.
https://reviews.llvm.org/D28425
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailm
compnerd added inline comments.
Comment at: src/cxa_new_delete.cpp:34
*/
-__attribute__((__weak__, __visibility__("default")))
+__attribute__((__weak__))
void *
EricWF wrote:
> Can we abstract this away to a `_LIBCXXABI_NEW_DELETE_VIS` macro?
Are `operator new`
dylanmckay updated this revision to Diff 83496.
dylanmckay added a comment.
Add tests
This also defines the '__AVR' and 'AVR' values for all devices, to match what
AVR-GCC does.
https://reviews.llvm.org/D28346
Files:
lib/Basic/Targets.cpp
test/CodeGen/avr/target-cpu-defines/atmega328p.c
mehdi_amini added inline comments.
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:910-912
// OptimizeNone wins over OptimizeForSize and MinSize.
F->removeFnAttr(llvm::Attribute::OptimizeForSize);
F->removeFnAttr(llvm::Attribute::MinSize);
chandler
dylanmckay updated this revision to Diff 83504.
dylanmckay added a comment.
Fix a few cases of unintentional switch fallthrough
https://reviews.llvm.org/D28344
Files:
lib/Basic/Targets.cpp
Index: lib/Basic/Targets.cpp
===
--- l
FTR, this wasn't changed with the renaming.
But, taking a look at it does seem that it is off by one. Well spotted!
On Thu, Jan 5, 2017 at 7:16 PM David Majnemer
wrote:
> On Thu, Jan 5, 2017 at 3:25 PM, Saleem Abdulrasool via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
>
> Author: compn
Author: compnerd
Date: Fri Jan 6 23:13:32 2017
New Revision: 291336
URL: http://llvm.org/viewvc/llvm-project?rev=291336&view=rev
Log:
system_error: correct ELAST emulation on Windows
ELAST should point to the last valid error string value. However,
`_sys_nerr` provides the number of elements in
dylanmckay updated this revision to Diff 83507.
dylanmckay added a comment.
Add tests for inline assembly constraints
https://reviews.llvm.org/D28344
Files:
lib/Basic/Targets.cpp
test/CodeGen/avr-inline-asm-constraints.c
Index: test/CodeGen/avr-inline-asm-constraints.c
dylanmckay marked an inline comment as done.
dylanmckay added inline comments.
Comment at: lib/Basic/Targets.cpp:8506
+ case 'N': // Integer constant (Range: -1)
+Info.setRequiresImmediate(-1);
+ case 'O': // Integer constant (Range: 8, 16, 24)
mehdi_amini updated this revision to Diff 83510.
mehdi_amini added a comment.
Rebase
https://reviews.llvm.org/D24688
Files:
clang/lib/CodeGen/BackendUtil.cpp
clang/lib/CodeGen/CodeGenModule.cpp
clang/test/CodeGen/nobuiltin.c
clang/test/CodeGenCUDA/flush-denormals.cu
clang/test/CodeGen
EricWF created this revision.
EricWF added reviewers: compnerd, smeenai, rnk, majnemer.
EricWF added a subscriber: cfe-commits.
This patch implements the following functions on Windows by forwarding to the
MSVCRT:
- `get_terminate()`
- `set_terminate()`
- `terminate()`
- `set_unexpected()`
- `ge
EricWF added a comment.
Adding cfe-commits
https://reviews.llvm.org/D28441
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
pcc added a comment.
Didn't we figure out in the end that this could be a function attribute instead?
https://reviews.llvm.org/D24688
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mehdi_amini added a comment.
In https://reviews.llvm.org/D24688#638835, @pcc wrote:
> Didn't we figure out in the end that this could be a function attribute
> instead?
We did? You wrote in PR30403: "I had a brief look at what it would take to have
a per-function TLI, and I'm not convinced th
Author: ericwf
Date: Sat Jan 7 00:09:12 2017
New Revision: 291337
URL: http://llvm.org/viewvc/llvm-project?rev=291337&view=rev
Log:
Explicitly specify MSVC mangling of iostream globals. Patch from Dave Lee
Modified:
libcxx/trunk/src/iostream.cpp
Modified: libcxx/trunk/src/iostream.cpp
URL:
smeenai added inline comments.
Comment at: CMakeLists.txt:43
+if (WIN32 AND NOT MINGW)
+ set(LIBCXX_TARGETING_WINDOWS ON)
+else()
Not the biggest fan of this name, since it's not obvious why MinGW shouldn't
count as targeting Windows. I thought of `LIBCXX_TARGE
smeenai accepted this revision.
smeenai added a comment.
This revision is now accepted and ready to land.
Hooray for Microsoft for putting all these in msvcrt (their **C** runtime
library) instead of msvcprt (their C++ runtime library), I guess :p
Comment at: src/exception.cpp
halyavin added inline comments.
Comment at: lib/CMakeLists.txt:108-109
+if (LIBCXX_TARGETING_WINDOWS)
+ add_compile_flags(/Zl)
+ add_link_flags(/nodefaultlib)
+ add_library_flags(ucrt) # Universal C runtime
smeenai wrote:
> These should be guarded under a chec
Wouldn't it be better to introduce a __libcpp_thread_nanosleep() API call
here?
I bumped into a similar issue with a custom thread implementation and have
a downstream patch like that.
Cheers,
/ Asiri
On 7 Jan 2017 2:59 a.m., "Saleem Abdulrasool via cfe-commits" <
cfe-commits@lists.llvm.org> w
101 - 189 of 189 matches
Mail list logo
