>>Do you see a reason this wouldn't work?
No, I do not see any.This is good.
>>so it's probably simplest to go ahead with mine.
Yes, thank you.
On Mon, Nov 2, 2020 at 9:48 PM Jason Merrill wrote:
>
> On 11/2/20 10:10 AM, kamlesh kumar wrote:
> > addressed jason comments.
> > no regression due to
addressed jason comments.
no regression due to this, tested on x86_64 linux.
On Tue, Oct 27, 2020 at 11:09 PM Jason Merrill wrote:
>
> On 10/22/20 1:31 PM, kamlesh kumar wrote:
> > Attaching the patch file.
> >
> > >>Instead of building a hash table, would it work to handle ambiguity by
> >
Attaching the patch file.
>>Instead of building a hash table, would it work to handle ambiguity by
>>checking whether one of the classes is a base of the other?
Fixing for cases like: struct B: A,A may not be cleaner this
way.
On Thu, Oct 22, 2020 at 3:23 AM Jason Merrill wrote:
>
> On 10/21/20
gcc/cp/ChangeLog
---
2020-10-21 Kamlesh Kumar
PR c++/97453
* pt.c (get_template_base): Implement DR2303,
Consider closest base while template
deduction when base of base also matches.
gcc/testsuite/ChangeLog
--
be used with the wi:: routines. If we ever put modes into rtx
integer constants, this should go away and then just pass an rtx
in. */
On 16/06/20 2:34 pm, Richard Sandiford wrote:
Thanks for doing this.
Kamlesh Kumar via Gcc-patches writes:
diff --git a/gcc/rtl.h b/gcc/rtl.h
index
Thank you all for the suggestions.
This is first patch where I have just defined a struct libcall_arg_t
which contains
three member rtx, machine_mode and a boolean unsigned_p and will be
used in passing args in
emit_library_[call/value] functions.
Once this patch is approved then i will create
This is first patch where I have just defined a struct libcall_arg_t which
contains
three member rtx, machine_mode and a boolean unsigned_p and will be used in
passing args in
emit_library_[call/value] functions.
Once this patch is approved then i will create second patch in which arg type
This is first patch where I have just defined a struct libcall_arg_t which
contains
three member rtx, machine_mode and a boolean unsigned_p and will be used in
passing args in
emit_library_[call/value] functions.
Once this patch is approved then i will create second patch in which arg type
> OTOH, you don't need to name Tuple at all... It should not *have* a
> constructor, since you declared it as class... But you can just use
> std::tuple here?
I thought of using std::tuple but it requires c++11 support.
I am not sure we always build gcc with c++11?
>
> >
Here is a discussion we did some time ago regarding the defect.
https://gcc.gnu.org/pipermail/gcc/2019-January/227834.html
please see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88877 for testcase
behavior.
We incorporating below Jakub's suggestion in this patch series.
Jakub wrote:
""
Yeah,
ping?
On Tue, May 19, 2020 at 5:32 PM kamlesh kumar
wrote:
> can someone look at the patch, please?
>
>
> On Wed, Apr 8, 2020 at 9:29 PM Jeff Law wrote:
>
>> On Mon, 2020-04-06 at 14:58 +0530, kamlesh kumar via Gcc-patches wrote:
>> > Hi Richard,
>> > He
can someone look at the patch, please?
On Wed, Apr 8, 2020 at 9:29 PM Jeff Law wrote:
> On Mon, 2020-04-06 at 14:58 +0530, kamlesh kumar via Gcc-patches wrote:
> > Hi Richard,
> > Here is a discussion we did some time ago
> > https://gcc.gnu.org/pipermail/gcc/201
added VERIFY in test and changed the template parameter naming.
diff --git a/libstdc++-v3/include/std/any b/libstdc++-v3/include/std/any
index 6b7e68f0e63..d350d0b2575 100644
--- a/libstdc++-v3/include/std/any
+++ b/libstdc++-v3/include/std/any
@@ -176,36 +176,23 @@
Added the fix for emplace.
diff --git a/libstdc++-v3/include/std/any b/libstdc++-v3/include/std/any
index 6b7e68f0e63..f35d90e548d 100644
--- a/libstdc++-v3/include/std/any
+++ b/libstdc++-v3/include/std/any
@@ -178,30 +178,17 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
/// Construct with a copy of
Thank you for reviewing.
without _Decay to decay_t in the constructor which takes inplace_type_t,
cases like this fails
auto a = std::any(std::in_place_type, 5);
for these constructors, standard does not say anything about
not-sameness checks with any.
On Fri, Apr 17, 2020, 10:59 PM kamlesh kumar
wrote:
> Fixes all this.
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92156
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91630
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90415
>
> On Fri, Apr 17, 2020 at 10:45 PM kamlesh kumar
> wrote:
> >
>
Fixes all this.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92156
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91630
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90415
On Fri, Apr 17, 2020 at 10:45 PM kamlesh kumar wrote:
>
> This patch corrects the requirement of 4,5 and 6th constructor
> As
This patch corrects the requirement of 4,5 and 6th constructor
As per https://en.cppreference.com/w/cpp/utility/any/any.
ChangeLog:
2020-04-17 Kamlesh Kumar
PR libstdc++/92156
* include/std/any (ans::any(_ValueType &&):: Remove
is_constructible.
Hi Richard,
Here is a discussion we did some time ago
https://gcc.gnu.org/pipermail/gcc/2019-January/227834.html
please see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88877 for more
info regarding the bug.
We incorporated below Jakub's suggestion in this patch.
Jakub wrote:
""
Yeah, all the
Segher,
Please provide your suggestion/thought on the fix.
On Mon, Mar 23, 2020 at 8:16 PM kamlesh kumar wrote:
>
> Attached patch fixes.
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88877.
> ChangeLog Entry.
>
> 2020-03-23 Kamlesh Kumar
>
> * rtl.h : Defined Tuple for bundling rtx,
Thanks Hans-Peter for reviewing.
Here is Formatted ChangeLog Entry.
2020-03-24 Kamlesh Kumar
* rtl.h: Defined Tuple for bundling rtx, mode and unsignedness
(default as 0), Added Extra argument (unsignedp)
in emit_library_call and emit_library_call_value.
* except.c: Likewise.
* explow.c:
Attached patch fixes.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88877.
ChangeLog Entry.
2020-03-23 Kamlesh Kumar
* rtl.h : Defined Tuple for bundling rtx, mode and
unsignedness default as 0
Added Extra argument (unsignedp) in emit_library_call and
emit_library_call_value.
22 matches
Mail list logo
