Re: [PATCH 0/4] Add aarch64-darwin support for off-stack trampolines

2021-12-02 Thread Iain Sandoe
> On 3 Dec 2021, at 03:12, Jeff Law wrote: > > > > On 11/22/2021 7:49 AM, Maxim Blinov wrote: >> Hi all, apologies for forgetting to add the cover letter. > No worries. I'd already assumed this was to support aarch64 trampolines on > darwin by having them live elsewere as managed

[Bug c++/87282] Invalid use of 'void', in unevaluated context

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87282 --- Comment #3 from Andrew Pinski --- (In reply to Andrew Pinski from comment #2) > Dup of bug 78173. I should say this was fixed in GCC 11+

[Bug c++/78173] Hard error subtracting pointers to incomplete type in SFINAE context

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78173 Andrew Pinski changed: What|Removed |Added CC||delrieutheo at gmail dot com ---

[Bug c++/87282] Invalid use of 'void', in unevaluated context

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87282 Andrew Pinski changed: What|Removed |Added Resolution|--- |DUPLICATE Status|NEW

[Bug c++/86619] Missed optimization opportunity with array aliasing

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86619 --- Comment #5 from Andrew Pinski --- I Noticed clang, ICC nor MSVC either handle this either. Note GCC is the only one which handles : int f(std::array & a, std::array & b) { a[0] = 1; b[0] = 1; return a[0]; }

[Bug c/103531] Propose compiler warning when ceil/ceilf used on integral value

2021-12-02 Thread eyalroz1 at gmx dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103531 --- Comment #4 from Eyal Rozenberg --- (In reply to Eric Gallager from comment #3) > -Wtraditional-conversion catches this: Well... you're technically right, but: 1. That is a much wider warning. If someone were to turn this on they would get

[Bug c++/87235] g++ doesn't implement sparse initialisation of arrays

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87235 Andrew Pinski changed: What|Removed |Added Keywords||diagnostic

Re: [power-ieee128] What should the math functions be annotated with?

2021-12-02 Thread Thomas Koenig via Gcc
On 01.12.21 21:54, Jakub Jelinek via Fortran wrote: Inside of libgfortran, I think it should depend on some macro defined in libgfortran.h. #if defined(__powerpc64__) && __BYTE_ORDER__ == __ORDER_LITTLE_ENDIAN__ \ && defined __GLIBC_PREREQ && __GLIBC_PREREQ (2, 32) then #define

[Bug c++/86598] Incorrect lexing of pp-numbers in C++11 due to hexfloat extension

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86598 --- Comment #4 from Andrew Pinski --- Here is a full testcase that still fails: template struct Units{int operator+(int);}; class Amp{}; Units operator"" _amp(unsigned long long d); int k = 4_amp+1; // incorrect error: no literal operator ""

[Bug preprocessor/23827] standard C++ should not have hex float preprocessing tokens

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=23827 Andrew Pinski changed: What|Removed |Added Target Milestone|--- |6.0

[Bug c++/86743] [c++98] Compilation failure when initializing POD structs containing constant member

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86743 Andrew Pinski changed: What|Removed |Added Summary|Compilation failure when|[c++98] Compilation failure

[PATCH] tree-optimization/103514 Missing XOR-EQ-AND Optimization

2021-12-02 Thread Navid Rahimi via Gcc-patches
Hi GCC community, This patch will add the missed pattern described in bug 103514 [1] to the match.pd. Tested on x86_64 Linux. tree-optimization/103514 Missing XOR-EQ-AND Optimization * match.pd (a & b) == (a ^ b) -> !(a | b): New optimization. * match.pd (a & b) ^ (a == b) ->

RE: [PATCH v2] tree-optimization/101186 - extend FRE with "equivalence map" for condition prediction

2021-12-02 Thread Di Zhao OS via Gcc-patches
I'm very sorry there seems to be encoding issue in the attachment in my last email. Attached is the new patch. Thanks, Di Zhao > -Original Message- > From: Di Zhao OS > Sent: Tuesday, November 16, 2021 1:24 AM > To: 'Richard Biener' > Cc: gcc-patches@gcc.gnu.org > Subject: RE: [PATCH

[Bug c++/79318] [DR 1726] conversion member function preceded with & is not marked as error

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79318 Andrew Pinski changed: What|Removed |Added Last reconfirmed||2021-12-03

Re: [PATCH] stddef.h: add support for musl typedef macro guards

2021-12-02 Thread Sören Tempel via Gcc-patches
Hi, Jeff Law wrote: > So what doesn't make sense here is how both stddef.h files get > included.  That's the core problem I think you need to resolve. The libgo/sysinfo.c file includes stddef.h (for which the GCC version in ginclude is used on my system) and stdlib.h which, on musl, causes an

[PATCH] [i386] Prefer INT_SSE_REGS for SSE_FLOAT_MODE_P in preferred_reload_class.

2021-12-02 Thread liuhongt via Gcc-patches
Hi: > Please also consider TARGET_INTER_UNIT_MOVES_TO_VEC and > TARGET_INTER_UNIT_MOVES_FROM_VEC. Here's updated patch. Also honor TARGET_INTER_UNIT_MOVES_TO/FROM_VEC and in preferred_{,out_}reload_class. Bootstrapped and regtested on x86_64-pc-linux-gnu{-m32\ -march=k8,\ -march=k8}. Ok?

[Bug c++/72777] broken error message with reference in constexpr arguments in c++11 mode

2021-12-02 Thread de34 at live dot cn via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72777 Jiang An changed: What|Removed |Added CC||de34 at live dot cn --- Comment #2 from

[Bug c++/99931] Unnamed `struct` defined with `using` having internal linkage instead of external, unlike `typedef`, yielding different semantics for two

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99931 Andrew Pinski changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug c++/61457] [C++11/14 only] Inaccurately refusing non-ODR usage of type with no linkage

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61457 Andrew Pinski changed: What|Removed |Added Summary|Inaccurately refusing |[C++11/14 only]

[Bug c++/60277] Bogus "inline function virtual ..." used but never defined

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60277 Andrew Pinski changed: What|Removed |Added Target Milestone|--- |9.3 Resolution|---

[Bug c++/60551] __attribute__((unused)) is ignored for 'static' declarations

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60551 Andrew Pinski changed: What|Removed |Added Summary|__attribute__((used)) is|__attribute__((unused)) is

[Bug rtl-optimization/103028] ICE in extract_constrain_insn, at recog.c:2670

2021-12-02 Thread aoliva at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103028 --- Comment #6 from Alexandre Oliva --- This will probably avoid the error. valid_insn_p checks the alternatives, and fails for the invalid cmpdi_ccu that we attempt to create. Conceivably, this could be avoided by narrowing down the

[Bug c++/98939] [C++23] Implement P1787R6 "Declarations and where to find them"

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98939 --- Comment #6 from Andrew Pinski --- Note I think this paper applies to C++20 too or at least part of it. >From CWG1291: [Accepted at the November, 2020 meeting as part of paper P1787R6 and moved to DR at the February, 2021 meeting.]

[Bug c++/69701] "v.operator T()" incorrectly parsed if "v.T()" present.

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69701 --- Comment #6 from Andrew Pinski --- http://www.open-std.org/jtc1/sc22/wg21/docs/cwg_defects.html#1291 [Accepted at the November, 2020 meeting as part of paper P1787R6 and moved to DR at the February, 2021 meeting.] So there is a defect

[Bug c++/69701] "v.operator T()" incorrectly parsed if "v.T()" present.

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69701 Andrew Pinski changed: What|Removed |Added Keywords|accepts-invalid | --- Comment #5 from Andrew Pinski ---

[PATCH v2] regrename: Skip renaming if instruction is noop move.

2021-12-02 Thread Jojo R via Gcc-patches
Skip renaming if instruction is noop move, and it will been removed for performance. gcc/ * regrename.c (find_rename_reg): Return satisfied regno if instruction is noop move. --- gcc/regrename.c | 5 + 1 file changed, 5 insertions(+) diff --git a/gcc/regrename.c

[Bug c++/69701] "v.operator T()" incorrectly parsed if "v.T()" present.

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69701 Andrew Pinski changed: What|Removed |Added Ever confirmed|0 |1 Last reconfirmed|

[Bug c++/69701] "v.operator T()" incorrectly parsed if "v.T()" present.

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69701 --- Comment #3 from Andrew Pinski --- ICC also rejects this for the same reason as GCC while both MSVC and clang accept it. Maybe there is a defect report about this.

[Bug rtl-optimization/103028] ICE in extract_constrain_insn, at recog.c:2670

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103028 --- Comment #5 from Andrew Pinski --- (In reply to Alexandre Oliva from comment #4) > Andrew, > > asm("":"=g"(tt):"g"(t)); > asm("":"=g"(ii):"g"(i)); > > Make it "0" for the inputs: > > asm("":"=g"(tt):"0"(t)); Oh yes sorry I did make

[Bug rtl-optimization/103028] ICE in extract_constrain_insn, at recog.c:2670

2021-12-02 Thread aoliva at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103028 --- Comment #4 from Alexandre Oliva --- Andrew, asm("":"=g"(tt):"g"(t)); asm("":"=g"(ii):"g"(i)); Make it "0" for the inputs: asm("":"=g"(tt):"0"(t)); and AFAICT if you "detach" the immediate constant, you won't get the bug. The

[Bug c/103531] Propose compiler warning when ceil/ceilf used on integral value

2021-12-02 Thread egallager at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103531 Eric Gallager changed: What|Removed |Added CC||egallager at gcc dot gnu.org

[Bug c++/68699] Wrong location_t due to template instance canonicalization when setting location_t values on expressions

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68699 Andrew Pinski changed: What|Removed |Added Status|UNCONFIRMED |NEW Ever confirmed|0

[Bug c++/48850] Bogus overflow in constant expression warning

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=48850 Andrew Pinski changed: What|Removed |Added See Also||https://gcc.gnu.org/bugzill

[PATCH v2] rs6000: Fix some issues in rs6000_can_inline_p [PR102059]

2021-12-02 Thread Kewen.Lin via Gcc-patches
Hi Segher, Thanks for the review! on 2021/11/30 上午12:57, Segher Boessenkool wrote: > Hi! > > On Wed, Sep 01, 2021 at 02:55:51PM +0800, Kewen.Lin wrote: >> This patch is to fix the inconsistent behaviors for non-LTO mode >> and LTO mode. As Martin pointed out, currently the function >>

[Bug c++/57239] cannot handle inner/nested class templates with non-type parameter packs that were declared in the outer/containing class

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57239 --- Comment #14 from Andrew Pinski --- clang accepts the reduced testcase in comment #1 but MSVC and ICC both reject it too. MSVC: (18): error C3201: the template parameter list for class template 'Foo' does not match the template parameter

[Bug rtl-optimization/93027] ICE: in match_reload, at lra-constraints.c:1060

2021-12-02 Thread aoliva at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93027 Alexandre Oliva changed: What|Removed |Added CC||aoliva at gcc dot gnu.org --- Comment

[PATCH v2] regrename: Skip renaming if instruction is noop move.

2021-12-02 Thread Jojo R via Gcc-patches
Skip renaming if instruction is noop move, and it will been removed for performance. gcc/ * regrename.c (find_rename_reg): Return satisfied regno if instruction is noop move. --- gcc/regrename.c | 5 + 1 file changed, 5 insertions(+) diff --git a/gcc/regrename.c

Re: [PATCH] rs6000: Fix some issues in rs6000_can_inline_p [PR102059]

2021-12-02 Thread Kewen.Lin via Gcc-patches
Hi Mike, on 2021/12/3 上午8:51, Michael Meissner wrote: > On Mon, Nov 29, 2021 at 10:57:12AM -0600, Segher Boessenkool wrote: >> Why are there OPTION_MASKs for separate P10 fusion types here, as well as >> MASK_P10_FUSION? > > Well going back in time, before we used rs6000_isa_flags, we used the

Re: [PATCH] Avoid expecting nonzero size for access none void* arguments [PR101751]

2021-12-02 Thread Jeff Law via Gcc-patches
On 11/24/2021 7:16 PM, Martin Sebor via Gcc-patches wrote: When the optional size-index argument to attribute index is omitted for a pointer, GCC expects the actual pointer argument to point to an object at least as big as its size implies, or at least one byte for void*.  This is done to

Re: [PATCH] correct handling of offsets in PHI expressions [PR103215]

2021-12-02 Thread Jeff Law via Gcc-patches
On 11/22/2021 4:54 PM, Martin Sebor via Gcc-patches wrote: In an effort to avoid false positives while still detecting certain out-of-bounds accesses the warning code that handles PHI nodes chooses the operand with the most space remaining as the one representative of the PHI.  That's not

Re: [PATCH 3/4] Add aarch64-linux support for off-stack trampolines

2021-12-02 Thread Jeff Law via Gcc-patches
On 11/13/2021 2:45 AM, Maxim Blinov wrote: Implement the __builtin_nested_func_ptr_{created,deleted} functions for the aarch64-linux platform. This serves to exercise the infrastructure added in libgcc (--enable-off-stack-trampolines) and gcc (-foff-stack-trampolines) in supporting off-stack

Re: [PATCH 2/4] Add x86_64-linux support for off-stack trampolines

2021-12-02 Thread Jeff Law via Gcc-patches
On 11/13/2021 2:45 AM, Maxim Blinov wrote: Implement the __builtin_nested_func_ptr_{created,deleted} functions for the x86_64-linux platform. This serves to exercise the infrastructure added in libgcc (--enable-off-stack-trampolines) and gcc (-foff-stack-trampolines) in supporting off-stack

Re: [PATCH 1/4] Generate off-stack nested function trampolines

2021-12-02 Thread Jeff Law via Gcc-patches
On 11/13/2021 2:45 AM, Maxim Blinov wrote: Add support for allocating nested function trampolines on an executable heap rather than on the stack. This is motivated by targets such as AArch64 Darwin, which globally prohibit executing code on the stack. The target-specific routines for

Re: [PATCH 0/4] Add aarch64-darwin support for off-stack trampolines

2021-12-02 Thread Jeff Law via Gcc-patches
On 11/22/2021 7:49 AM, Maxim Blinov wrote: Hi all, apologies for forgetting to add the cover letter. No worries.  I'd already assumed this was to support aarch64 trampolines on darwin by having them live elsewere as managed entities. The motivation of this work is to provide (limited)

Re: [PATCH 3/5] gcc: Add --nostdlib++ option

2021-12-02 Thread Jeff Law via Gcc-patches
On 10/28/2021 10:41 AM, Richard Purdie via Gcc-patches wrote: On Wed, 2021-10-27 at 22:56 +0200, Bernhard Reutner-Fischer wrote: On Wed, 27 Oct 2021 21:05:03 +0100 Richard Purdie via Gcc-patches wrote: OpenEmbedded/Yocto Project builds libgcc and the other gcc runtime libraries separately

Re: [PATCH 3/5] gcc: Add --nostdlib++ option

2021-12-02 Thread Jeff Law via Gcc-patches
On 10/28/2021 10:39 AM, Richard Purdie wrote: On Thu, 2021-10-28 at 08:51 -0600, Jeff Law wrote: On 10/27/2021 2:05 PM, Richard Purdie via Gcc-patches wrote: OpenEmbedded/Yocto Project builds libgcc and the other gcc runtime libraries separately from the compiler and slightly differently to

Re: [PATCH 2/5] gcc: Fix "argument list too long" from install-plugins

2021-12-02 Thread Jeff Law via Gcc-patches
On 10/27/2021 2:05 PM, Richard Purdie via Gcc-patches wrote: When building in longer build paths (200+ characters), the "echo $(PLUGIN_HEADERS)" from the install-plugins target would cause an "argument list too long error" on some systems. Avoid this by calling make's sort function on the

Re: [PATCH] build: Implement --with-multilib-list for avr target

2021-12-02 Thread Jeff Law via Gcc-patches
On 6/7/2021 1:30 AM, Matt Jacobson via Gcc-patches wrote: gcc/ChangeLog: 2020-06-07 Matt Jacobson * config.gcc: For the AVR target, populate TM_MULTILIB_CONFIG. * config/avr/genmultilib.awk: Add ability to filter generated multilib list. * config/avr/t-avr:

[PATCH] Improve AutoFDO count propagation algorithm

2021-12-02 Thread Eugene Rozenfeld via Gcc-patches
When a basic block A has been annotated with a count and it has only one successor (or predecessor) B, we can propagate the A's count to B. The algorithm without this change could leave B without an annotation if B had other unannotated predecessors (or successors). For example, in the test case I

Re: [PATCH v3] MIPS: R6: load/store can process unaligned address

2021-12-02 Thread Jeff Law via Gcc-patches
On 10/11/2021 4:42 AM, YunQiang Su wrote: MIPS release 6 requires the lw/ld/sw/sd can work with unaligned address, while it can be implemented by full hardware or trap Since it doesn't have to be fully done by hardware, we add a pair of options -m(no-)unaligned-access. Kernels may need them.

Re: PING^1 [PATCH] Add TARGET_IFUNC_REF_LOCAL_OK

2021-12-02 Thread Jeff Law via Gcc-patches
On 7/6/2021 5:47 AM, H.J. Lu via Gcc-patches wrote: On Sat, Jun 19, 2021 at 7:10 PM H.J. Lu wrote: 1. On some targets, like PowerPC, reference to ifunc function resolver must be non-local so that compiler will properly emit PLT call. Add TARGET_IFUNC_REF_LOCAL_OK to allow binding indirect

[Bug c++/83756] [9/10/11/12 Regression] gcc ICE in gimplify_expr on boost::call_once (-fms-extensions)

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83756 Andrew Pinski changed: What|Removed |Added CC||lh_mouse at 126 dot com --- Comment #13

[Bug c++/85776] ICE when compiling `boost::thread` with `-std=c++11` on mingw-w64

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85776 Andrew Pinski changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug c++/94404] [meta-bug] C++ core issues

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94404 Bug 94404 depends on bug 63217, which changed state. Bug 63217 Summary: template conversion operator returning const reference not used for conversion in some cases https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63217 What|Removed

[Bug c++/61663] [DR 976] Deduction for const T& conversion functions

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61663 Andrew Pinski changed: What|Removed |Added CC||rs2740 at gmail dot com --- Comment #3

[Bug c++/63217] template conversion operator returning const reference not used for conversion in some cases

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63217 Andrew Pinski changed: What|Removed |Added Resolution|--- |DUPLICATE Status|NEW

[Bug c++/63217] template conversion operator returning const reference not used for conversion in some cases

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63217 --- Comment #9 from Andrew Pinski --- Other related ones: http://www.open-std.org/jtc1/sc22/wg21/docs/cwg_defects.html#493 http://www.open-std.org/jtc1/sc22/wg21/docs/cwg_defects.html#322

[Bug c++/63217] template conversion operator returning const reference not used for conversion in some cases

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63217 Andrew Pinski changed: What|Removed |Added Blocks||94404 Alias|

[Bug c++/63217] template conversion operator returning const reference not used for conversion in some cases

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63217 --- Comment #7 from Andrew Pinski --- ICC also rejects this for the same reason as GCC. MSVC and clang both accept it though. Could there be a defect report for this?

[Bug c++/63217] template conversion operator returning const reference not used for conversion in some cases

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63217 Andrew Pinski changed: What|Removed |Added CC||vz at quantitativesystems dot com ---

[Bug c++/79035] Rejects valid code due to failure to use user const conversion template operator

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79035 Andrew Pinski changed: What|Removed |Added Resolution|--- |DUPLICATE

[Bug c++/79035] Rejects valid code due to failure to use user const conversion template operator

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79035 --- Comment #1 from Andrew Pinski --- ICC also rejects this for the same reason as GCC. MSVC and clang both accept it though.

[Bug target/102376] [aarch64] using target("sve") attribute without a + is not very helpful on what is wrong

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102376 Andrew Pinski changed: What|Removed |Added Status|ASSIGNED|RESOLVED Target Milestone|---

[Bug c++/102647] Wrong caret location for 'attribute directive ignored' warning

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102647 Andrew Pinski changed: What|Removed |Added See Also||https://gcc.gnu.org/bugzill

[Bug tree-optimization/103536] New: Suboptimal codegen for && and || combination.

2021-12-02 Thread navidr at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103536 Bug ID: 103536 Summary: Suboptimal codegen for && and || combination. Product: gcc Version: unknown Status: UNCONFIRMED Severity: normal Priority: P3

[Bug c++/66901] Segmentation fault in compiler instead of error message for ill-formed program with namespace alias and qualified definition

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66901 Andrew Pinski changed: What|Removed |Added Known to work||8.1.0, 9.1.0 Ever confirmed|0

[Bug c++/66878] Segmentation fault when compiling

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66878 Andrew Pinski changed: What|Removed |Added Known to work||11.1.0 Status|RESOLVED

[Bug c++/66934] Compiler accepting ill-formed program with extern variable declarations and using-declaration

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66934 Andrew Pinski changed: What|Removed |Added Known to work||8.1.0 Last reconfirmed|

[Bug c++/95074] Function found via ADL when it should not

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95074 Andrew Pinski changed: What|Removed |Added Target Milestone|--- |11.0

[Bug c++/71755] friend function may not be defined inside a class using a qualified name but GCC allows that

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71755 Andrew Pinski changed: What|Removed |Added Target Milestone|--- |10.0

Re: [PATCH] rs6000: Fix some issues in rs6000_can_inline_p [PR102059]

2021-12-02 Thread Michael Meissner via Gcc-patches
On Mon, Nov 29, 2021 at 10:57:12AM -0600, Segher Boessenkool wrote: > Why are there OPTION_MASKs for separate P10 fusion types here, as well as > MASK_P10_FUSION? Well going back in time, before we used rs6000_isa_flags, we used the default flag word for MASK arguments. Unfortunately, the

[Bug c++/53402] [C++11] non-inline namespace can be wrongly re-opened as inline

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53402 Andrew Pinski changed: What|Removed |Added See Also||https://gcc.gnu.org/bugzill

[Bug c++/79766] inline namespace and ::

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79766 Andrew Pinski changed: What|Removed |Added Target Milestone|--- |8.0

[Bug c++/66935] Compiler rejects well-formed program with local extern variable declaration and using-declaration

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66935 Andrew Pinski changed: What|Removed |Added Last reconfirmed||2021-12-03 Status|RESOLVED

[Bug c++/66934] Compiler accepting ill-formed program with extern variable declarations and using-declaration

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66934 --- Comment #9 from Andrew Pinski --- (In reply to Anders Granlund from comment #4) > I'm pulling in this test case from my related bug report. > > int x = 1; > > int main() { > extern int x; > using ::x; > } This is

[Bug c++/66934] Compiler accepting ill-formed program with extern variable declarations and using-declaration

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66934 Andrew Pinski changed: What|Removed |Added See Also||https://gcc.gnu.org/bugzill

[Bug c++/71886] Incorrect error on operator() being an member in template

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71886 Andrew Pinski changed: What|Removed |Added Resolution|--- |INVALID Status|UNCONFIRMED

[Bug c++/71841] variadic template can't cast to base class

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71841 Andrew Pinski changed: What|Removed |Added Status|WAITING |RESOLVED Resolution|---

[Bug c++/70446] g++: internal compiler error: Killed (program cc1plus), probably related to vectors

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70446 Andrew Pinski changed: What|Removed |Added Resolution|--- |WORKSFORME

[Bug c++/61587] Line number of code that triggers "use of deleted function" error is absent from error message

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61587 Andrew Pinski changed: What|Removed |Added Severity|normal |enhancement Last

[Bug c++/103534] [12 regression] Spurious -Wstringop-overflow warning with std::string concatencation

2021-12-02 Thread msebor at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103534 --- Comment #4 from Martin Sebor --- Yes, the warning does disappear when malloc() and free() are used instead of operator new and delete. foo() also ends up much better optimized, even at -O1: __attribute__((abi_tag ("cxx11"))) struct string

Re: [PATCH] rs6000: testsuite: Add rop_ok effective-target function

2021-12-02 Thread Segher Boessenkool
Hi! On Thu, Nov 11, 2021 at 04:12:08PM -0600, Peter Bergner wrote: > This patch adds a new effective-target function that tests whether > it is safe to emit the ROP-protect instructions and updates the > ROP test cases to use it. > > Segher, as we discussed offline, this uses the double [] which

Re: [PATCH] libiberty rust-demangle, ignore .suffix

2021-12-02 Thread Eduard-Mihai Burtescu
On Fri, Dec 3, 2021, at 00:07, Mark Wielaard wrote: > Hi Eddy, > > On Thu, Dec 02, 2021 at 07:35:17PM +0200, Eduard-Mihai Burtescu wrote: >> On Thu, Dec 2, 2021, at 19:17, Mark Wielaard wrote: >> > Rust v0 symbols can have a .suffix because if compiler transformations. >> >> For some context, the

[PATCH v2] c++: Fix for decltype(auto) and parenthesized expr [PR103403]

2021-12-02 Thread Marek Polacek via Gcc-patches
On Wed, Dec 01, 2021 at 11:16:27PM -0500, Jason Merrill wrote: > On 12/1/21 10:16, Marek Polacek wrote: > > In r11-4758, I tried to fix this problem: > > > >int & = 0; > >decltype(auto) j = i; // should behave like int & = i; error > > > > wherein do_auto_deduction was getting confused

[Bug middle-end/103530] ICE: in execute, at gimple-harden-conditionals.cc:397 with -O -fharden-compares

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103530 --- Comment #2 from Andrew Pinski --- gcc_checking_assert (TREE_CODE (TREE_TYPE (lhs)) == BOOLEAN_TYPE); This assert is incorrect because useless_type_conversion_p has the following code: /* If both the inner and outer types are

[Bug c++/103534] [12 regression] Spurious -Wstringop-overflow warning with std::string concatencation

2021-12-02 Thread redi at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103534 --- Comment #3 from Jonathan Wakely --- (In reply to Andrew Pinski from comment #2) > I also noticed that with -std=c++17 (or before), the constructor for > std::string is not inlined fully for both std::string objects which avoids > the false

Re: [PATCH] libiberty rust-demangle, ignore .suffix

2021-12-02 Thread Mark Wielaard
Hi, On Fri, Dec 03, 2021 at 06:58:36AM +1100, Nicholas Nethercote wrote: > On Fri, 3 Dec 2021 at 04:17, Mark Wielaard wrote: > > > > * rust-demangle.c (rust_demangle_callback): Ignore everything > > after '.' char in sym for v0. > > > > I just applied this change to Valgrind's

[PATCH] rs6000: Fix use of wrong enum for built-in function code.

2021-12-02 Thread Bill Schmidt via Gcc-patches
Hi! I discovered this bug while working on patches to remove the old built-ins infrastructure. I missed a spot in converting from the rs6000_builtins enum to the rs6000_gen_builtins enum. This fixes it. The fix is technically not right if new_builtins_are_enabled were to be set to zero, but

[Bug tree-optimization/97711] Failure to optimise "x & 1 ? x - 1 : x" to "x & -2"

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97711 Andrew Pinski changed: What|Removed |Added Severity|normal |enhancement

[Bug tree-optimization/97711] Failure to optimise "x & 1 ? x - 1 : x" to "x & -2"

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97711 Andrew Pinski changed: What|Removed |Added CC||jengelh at inai dot de --- Comment #6

[Bug tree-optimization/103535] [missed optimization] remainder-of-2 with subtract-1

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103535 Andrew Pinski changed: What|Removed |Added Resolution|--- |DUPLICATE Status|NEW

gcc-9-20211202 is now available

2021-12-02 Thread GCC Administrator via Gcc
Snapshot gcc-9-20211202 is now available on https://gcc.gnu.org/pub/gcc/snapshots/9-20211202/ and on various mirrors, see http://gcc.gnu.org/mirrors.html for details. This snapshot has been generated from the GCC 9 git branch with the following options: git://gcc.gnu.org/git/gcc.git branch

[Bug tree-optimization/103535] [missed optimization] remainder-of-2 with subtract-1

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103535 Andrew Pinski changed: What|Removed |Added See Also||https://gcc.gnu.org/bugzill

Re: [PATCH] rs6000: Builtins test changes for test_fpscr_[d]rn_builtin_error.c

2021-12-02 Thread Segher Boessenkool
On Thu, Dec 02, 2021 at 10:43:24AM -0600, Bill Schmidt wrote: > The new built-in infrastructure is now enabled! Congratulations, and thanks for all the work! Segher

[Bug target/103271] ICE in assign_stack_temp_for_type with -ftrivial-auto-var-init=pattern and VLAs and -mno-strict-align on riscv64

2021-12-02 Thread qinzhao at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103271 --- Comment #17 from qinzhao at gcc dot gnu.org --- (In reply to Richard Biener from comment #12) > diff --git a/gcc/internal-fn.c b/gcc/internal-fn.c > index 6ac3460d538..08f94b7a17a 100644 > --- a/gcc/internal-fn.c > +++ b/gcc/internal-fn.c >

[Bug tree-optimization/103535] [missed optimization] remainder-of-2 with subtract-1

2021-12-02 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103535 Andrew Pinski changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |pinskia at gcc dot gnu.org

[Bug tree-optimization/103535] New: [missed optimization] remainder-of-2 with subtract-1

2021-12-02 Thread jengelh at inai dot de via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103535 Bug ID: 103535 Summary: [missed optimization] remainder-of-2 with subtract-1 Product: gcc Version: 11.2.1 Status: UNCONFIRMED Severity: normal Priority: P3

Re: [PATCH] target: [PR102941] Fix inline-asm flags with non-REG_P output

2021-12-02 Thread Andrew Pinski via Gcc-patches
On Tue, Oct 26, 2021 at 2:45 AM apinski--- via Gcc-patches wrote: > > From: Andrew Pinski > > So the problem here is that arm_md_asm_adjust would > just create a set directly to the output memory which is wrong. > It needs to output to a temp register first and then do a > move. > > OK?

Re: [PATCH v2] [AARCH64] Fix PR target/103100 -mstrict-align and memset on not aligned buffers

2021-12-02 Thread Andrew Pinski via Gcc-patches
On Thu, Nov 18, 2021 at 5:55 PM apinski--- via Gcc-patches wrote: > > From: Andrew Pinski > > The problem here is that aarch64_expand_setmem does not change the alignment > for strict alignment case. This is a simplified patch from what I had > previously. > So constraining copy_limit to the

[Bug middle-end/103483] context-sensitive ranges change triggers stringop-overread

2021-12-02 Thread msebor at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103483 Martin Sebor changed: What|Removed |Added Keywords|missed-optimization | --- Comment #10 from Martin Sebor ---

  1   2   3   4   >