Author: rsmith
Date: Sun Jan 1 20:42:17 2017
New Revision: 290808
URL: http://llvm.org/viewvc/llvm-project?rev=290808&view=rev
Log:
DR1391: Check for implicit conversion sequences for non-dependent function
template parameters between deduction and substitution. The idea is to accept
as many case
Thanks both of you, r290807.
On 1 January 2017 at 07:28, Erik Pilkington
wrote:
>
> > On Dec 31, 2016, at 4:41 PM, Richard Smith via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
> >
> > Author: rsmith
> > Date: Sat Dec 31 15:41:23 2016
> > New Revision: 290792
> >
> > URL: http://llvm.org/
Author: rsmith
Date: Sun Jan 1 20:38:22 2017
New Revision: 290807
URL: http://llvm.org/viewvc/llvm-project?rev=290807&view=rev
Log:
Address post-commit review comments.
Modified:
cfe/trunk/include/clang/Basic/LangOptions.def
cfe/trunk/lib/Sema/SemaTemplateDeduction.cpp
Modified: cfe/tru
majnemer added inline comments.
Comment at: src/typeinfo.cpp:28-29
+#if defined(_WIN64)
+ static constexpr const size_t fnv_offset_basis = 14695981039346656037;
+ static constexpr const size_t fnv_prime = 10995116282110;
+#else
majnemer wrote:
> Why make these
mgorny created this revision.
mgorny added reviewers: rsmith, t.p.northover, hfinkel, EricWF.
mgorny added a subscriber: cfe-commits.
Correct the logic used to set `ATOMIC_*_LOCK_FREE` preprocessor macros not
to rely on the ABI alignment of types. Instead, just assume all those
types are aligned c
compnerd created this revision.
compnerd added reviewers: EricWF, mclow.lists, majnemer, rnk.
compnerd added subscribers: smeenai, kastiglione, cfe-commits.
compnerd set the repository for this revision to rL LLVM.
The RTTI structure is different on Windows when building under MS ABI.
Update the d
Very close! Comment below:
On Jan 1, 2017, at 5:04 PM, Saleem Abdulrasool via cfe-commits
wrote:
>
> Author: compnerd
> Date: Sun Jan 1 16:04:38 2017
> New Revision: 290806
>
> URL: http://llvm.org/viewvc/llvm-project?rev=290806&view=rev
> Log:
> chrono: address post commit comments from How
compnerd created this revision.
compnerd added reviewers: EricWF, mclow.lists.
compnerd added subscribers: smeenai, cfe-commits.
compnerd set the repository for this revision to rL LLVM.
When building without libc++abi and libcxxrt, provide the definition for
the std::type_info destructor.
Repos
compnerd created this revision.
compnerd added reviewers: EricWF, mclow.lists, majnemer.
compnerd added subscribers: smeenai, cfe-commits.
compnerd set the repository for this revision to rL LLVM.
Because the class is marked as `dllexport` and instantiated in
`algorithm.cpp` and is strongly define
Thanks for the review on this! I hope you wont mind taking a look at SVN
r290806 as well which tries to address the comments.
On Sun, Jan 1, 2017 at 1:09 PM, Howard Hinnant
wrote:
> On Jan 1, 2017, at 3:20 PM, Saleem Abdulrasool via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
> >
> > Aut
Author: compnerd
Date: Sun Jan 1 16:04:36 2017
New Revision: 290805
URL: http://llvm.org/viewvc/llvm-project?rev=290805&view=rev
Log:
chrono: address post-commit comments from majnemer
Correct style to match libc++ style as pointed out by David Majnemer on
IRC. NFC.
Modified:
libcxx/trunk/
Author: compnerd
Date: Sun Jan 1 16:04:38 2017
New Revision: 290806
URL: http://llvm.org/viewvc/llvm-project?rev=290806&view=rev
Log:
chrono: address post commit comments from Howard
Drawing some inspiration from code from Bill O'Neal as pointed out by
Howard, rework the code to avoid an overflo
On Jan 1, 2017, at 3:20 PM, Saleem Abdulrasool via cfe-commits
wrote:
>
> Author: compnerd
> Date: Sun Jan 1 14:20:41 2017
> New Revision: 290803
>
> URL: http://llvm.org/viewvc/llvm-project?rev=290803&view=rev
> Log:
> chrono: implement a Windows version of system_clock::now
>
> system_clock
Author: compnerd
Date: Sun Jan 1 14:20:41 2017
New Revision: 290803
URL: http://llvm.org/viewvc/llvm-project?rev=290803&view=rev
Log:
chrono: implement a Windows version of system_clock::now
system_clock::now is not entirely straight forward on Windows, which
does not have a clock_gettime functi
Author: compnerd
Date: Sun Jan 1 14:20:38 2017
New Revision: 290801
URL: http://llvm.org/viewvc/llvm-project?rev=290801&view=rev
Log:
build: dont detect libraries for Windows
Hard code the defaults for Windows for the time being. The checks
really are always going to return the same value. Tec
Author: compnerd
Date: Sun Jan 1 14:20:36 2017
New Revision: 290800
URL: http://llvm.org/viewvc/llvm-project?rev=290800&view=rev
Log:
build: differentiate between building for and on Windows
This is necessary to support cross-compiling a Windows libc++ from
Linux. The CMAKE_SYSTEM_HOST_NAME tel
Author: compnerd
Date: Sun Jan 1 14:20:43 2017
New Revision: 290804
URL: http://llvm.org/viewvc/llvm-project?rev=290804&view=rev
Log:
chrono: give Windows a steady_clock
Provide a definition for a steady monotonic clock by wrapping
QueryPerformanceCounter.
Modified:
libcxx/trunk/src/chrono.
Author: compnerd
Date: Sun Jan 1 14:20:40 2017
New Revision: 290802
URL: http://llvm.org/viewvc/llvm-project?rev=290802&view=rev
Log:
build: further improve flags handling for cl
This allows us to build with cl (or rather clang-cl) by using the
correct spelling for `-include` (`/FI` for cl). cl
Author: compnerd
Date: Sun Jan 1 13:16:02 2017
New Revision: 290799
URL: http://llvm.org/viewvc/llvm-project?rev=290799&view=rev
Log:
CodeGen: update comment about RTTI field
The MS ABI RTTI has a reserved field which is used as a cache for the
demangled name. It must be zero-initialized, which
amaiorano added inline comments.
Comment at: lib/Format/Format.cpp:1900
- if (!getPredefinedStyle(FallbackStyle, Style.Language, &Style)) {
-llvm::errs() << "Invalid fallback style \"" << FallbackStyle
- << "\" using LLVM style\n";
-return Style;
- }
+
> On Dec 31, 2016, at 4:41 PM, Richard Smith via cfe-commits
> wrote:
>
> Author: rsmith
> Date: Sat Dec 31 15:41:23 2016
> New Revision: 290792
>
> URL: http://llvm.org/viewvc/llvm-project?rev=290792&view=rev
> Log:
> [c++17] Implement P0522R0 as written. This allows a template template argum
djasper created this revision.
djasper added a reviewer: rsmith.
djasper added a subscriber: cfe-commits.
In many translation units I have tried, the calls to isIgnored() removed in
this patch are more expensive than doing the analysis that is behind it. The
speed-up in translation units I have
djasper created this revision.
djasper added a reviewer: rsmith.
djasper added a subscriber: cfe-commits.
Herald added a subscriber: sanjoy.
In many translation units I have tested, many of the calls to
DiagnosticsEngine::GetDiagStatePointForLoc are for source locations before the
first diag sta
23 matches
Mail list logo
