On 12/12/18 9:56 AM, Paolo Carlini wrote:
Hi,
it seems we can easily improve the location of this - not so uncommon in
novice code - error to point to the 'static' keyword.
OK.
Jason
On Wed, Dec 12, 2018 at 01:47:02PM -0600, Peter Bergner wrote:
> Libitm on POWER hardware looks for the "htm" bit in AT_HWCAP2 to determine
> whether it can use HTM when executing code within __transaction_atomic
> code blocks. However, on newer hardware and kernels, the "htm" bit is no
> longer s
On 12/11/18 9:41 PM, Sandra Loosemore wrote:
On 12/5/18 10:14 AM, Martin Sebor wrote:
On 12/4/18 8:49 PM, Sandra Loosemore wrote:
What is the "it" referenced in the user's questions you quoted? The
const/pure attributes? Those are function attributes. The text you
are adding is in the type
The attached patch committed in r267065 corrects a few minor
typos in the description of attribute copy in the manual.
Martin
gcc/ChangeLog:
* doc/extend.texi (attribute copy): Fix typos.
Index: gcc/doc/extend.texi
===
--- gcc/doc
Hello,
we hit a bug today while cross-compiling a C++ program with mingw32:
if random_shuffle or random_sample_n are called with a sequence of
elements whose length is higher than RAND_MAX, the functions don't
behave as expected because they ignore elements beyond RAND_MAX. This
does not happen of
On 12/11/18 5:55 PM, David Malcolm wrote:
PR c++/88375 reports that errors relating to invalid conversions in
initializations are reported at unhelpfully vague locations, as in
e.g.:
enum struct a : int {
one, two
};
struct foo {
int e1, e2;
a e3;
} arr[] = {
{ 1, 2, a::one },
{
Hi all,
I have this patch since a longer time in my tree. No obvious fallout
visible.
I'm going to commit this patch in the next days if no one objects.
TIA,
Andreas
2018-12-12 Andreas Tobler
* config.gcc: Enable TARGET_DEFAULT_ASYNC_UNWIND_TABLES on
aarch64*-*-freebsd*.
Hi Martin,
> Does a non-executable stack actually improve security?
Absolutely, it's like closing your front door rather than just leave it open
for anyone.
> For the alternative implementation using (custom) function
> descriptors (-fno-trampolines) the static chain becomes
> part of the ABI or
Hi Wilco,
Am Mittwoch, den 12.12.2018, 22:04 + schrieb Wilco Dijkstra:
> Hi Martin,
>
> > Does a non-executable stack actually improve security?
>
> Absolutely, it's like closing your front door rather than just leave it open
> for anyone.
The question is whether it is like closing the fro
Hi!
The following patch fixes __builtin_is_constant_evaluated and
__builtin_constant_p handling during static_assert evaluation.
finish_static_assert calls fold_non_dependent_expr and complains if the
result is not a constant expression, instead of requiring a constant
expression, which causes __b
Hi!
The following patch adds =*k, *km alternatives to the various
zero_extend?i?i2 patterns, because kmov* instructions actually zero extend
the destination to 64-bits. I've kept * for the alternatives because that
is what is used e.g. in *mov[sd]i_internal patterns (probably to make it
less like
Hi!
The following patch uses a new mode iterator to avoid 2 almost same
vptestm and 2 almost same vptestnm patterns, and adds patterns that zero
extend those results to wider k registers, because that is what the
instructions actually do.
Bootstrapped/regtested on x86_64-linux and i686-linux, ok
Hi!
As mentioned in the PR, we can use kxor? %kN, %kN, %kN or
kxnor? %kN, %kN, %kN to set %kN to 0 or -1, instead of
setting a GPR to that and moving to the mask register.
Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
2018-12-12 Jakub Jelinek
PR target/88465
Hi!
We support either the AVX2 gather loads (128-bit or 256-bit, using masks in
vector registers), both conditional and unconditional, but AVX512F gather
loads only unconditional. The problem was that tree-vect-stmts.c didn't
have code to deal with the integral masktype argument the builtins have
This patch by Lynn Boger increases the time to wait for signals to be
delivered in the TestAtomicStop test in the os/signal package. When
running gccgo tests on ppc64 or ppc64le, there are intermittent
failures in this test because the wait time is too small.
Bootstrapped and ran os/signal test on
Hi!
The following patch makes array or derived type module parameters from other
TUs predetermined shared, so that they aren't rejected with default(none) or
copied with default(firstprivate). I went through all the spots in the
Fortran FE that set TREE_READONLY, most of them are on PARM_DECLs (t
POSIX requires snprintf to fail with EOVERFLOW when the specified
bound exceeds INT_MAX. This requirement conflicts with the C
requirements on valid calls to the function and isn't universally
implemented (e.g., Glibc doesn't seem to follow it, and GCC has
historically not paid heed to it either)
On Wed, 2018-12-12 at 11:40 -0600, Segher Boessenkool wrote:
> On Tue, Dec 11, 2018 at 10:35:00AM -0500, David Malcolm wrote:
> > On Mon, 2018-12-10 at 22:47 +, Segher Boessenkool wrote:
> >
> > [...]
> >
> > > diff --git a/gcc/c/c-parser.c b/gcc/c/c-parser.c
> > > index 121a91c..652e53c 1006
On Wed, Dec 12, 2018 at 8:10 AM Cherry Zhang wrote:
>
> Thank you, Matthias!
>
> From the log, essentially all the tests aborted. The only place the new code
> can cause abort on all programs that I can think of is in the runtime startup
> code, probestackmaps, which calls value_size, which abor
On Wed, Dec 12, 2018 at 06:23:08PM -0500, David Malcolm wrote:
> On Wed, 2018-12-12 at 11:40 -0600, Segher Boessenkool wrote:
> > > Is there test coverage for these errors and notes?
> >
> > Yes, in this same patch. The error, that is; I have no idea how to
> > test
> > the note, and that belongs
On 12/12/18 2:52 PM, Segher Boessenkool wrote:
>> +/* This is a fairly new feature bit, so handle it not being defined. */
>> +#ifndef PPC_FEATURE2_HTM_NO_SUSPEND
>> +# define PPC_FEATURE2_HTM_NO_SUSPEND 0
>> +#endif
>
> Doing it this way can be pretty surprising for users not aware you defined
>
On Wed, Dec 12, 2018 at 07:30:08PM +1030, Alan Modra wrote:
> On Wed, Dec 12, 2018 at 08:43:41AM +0100, Rasmus Villemoes wrote:
> > diff --git a/libgcc/config/rs6000/tramp.S b/libgcc/config/rs6000/tramp.S
> > index 637f4510146..a9f0f3826dc 100644
> > --- a/libgcc/config/rs6000/tramp.S
> > +++ b/lib
Hi!
On Wed, Dec 12, 2018 at 11:52:56AM +, Andrew Stubbs wrote:
>
> @@ -9032,9 +9040,16 @@ proc check_effective_target_cet { } {
> } "-O2" ]
> }
>
> +
> # Return 1 if target supports floating point "infinite"
> proc check_effective_target_inf { } {
Stray newline?
Segher
> On Dec 12, 2018, at 5:12 PM, Uecker, Martin
> wrote:
>> ...
>> I've not seen such an alternative implementation (-fno-trampolines is
>> ignored on all targets I tried),
>
> It was implemented for Ada. But here is a patch to also
> activate it for C:
>
> https://gcc.gnu.org/ml/gcc-patches/2
On Wed, Dec 12, 2018 at 12:09:19PM +, Jozef Lawrynowicz wrote:
> Compilation of gcc.dg/pr85180.c and gcc.dg/pr87985.c times out after 5 minutes
> for msp430 with -mlarge.
>
> nonzero_bits1 (from rtlanal.c), recurses many times for each reg
> because reg_nonzero_bits_for_combine (combine.c) nev
Hi,
This patch calls update_max_bb_count even if autofdo counts are all zeros,
otherwise it would trigger ICE because of mismatch between basic blocks'
count (all autofdo::zero) and cfun->cfg->max_count (guessed::non_zero).
For functions with all autofdo::zero counts, we should improve by restoring
Hi,
This patch skips generating histogram value for internal function call in
autofdo,
otherwise it would trigger ICE with following patch re-enabling indirect call
value
profile transformation. I think this patch is actually needed for GCC-6 on
which
indirect call value profile is not disabled
Hi,
Due to ICE and mal-functional bugs, indirect call value profile transformation
is disabled on GCC-7/8/trunk. This patch restores the transformation. The
main issue is AutoFDO should store cgraph_node's profile_id of callee func in
the first histogram value's counter, rather than pointer to c
Hi!
On Mon, Dec 10, 2018 at 09:56:46PM -0700, Jeff Law wrote:
> Note that split-path-5 has the same basic structure. A half-diamond
> with a single statement in the middle block that should be trivially
> if-convertable if profitable. So I adjusted that testcase.
The split-path-5.c testcase now
On Wed, Dec 12, 2018 at 11:34 PM Jakub Jelinek wrote:
>
> Hi!
>
> The following patch adds =*k, *km alternatives to the various
> zero_extend?i?i2 patterns, because kmov* instructions actually zero extend
> the destination to 64-bits. I've kept * for the alternatives because that
> is what is use
On Wed, Dec 12, 2018 at 11:36 PM Jakub Jelinek wrote:
>
> Hi!
>
> The following patch uses a new mode iterator to avoid 2 almost same
> vptestm and 2 almost same vptestnm patterns, and adds patterns that zero
> extend those results to wider k registers, because that is what the
> instructions actu
On Wed, 12 Dec 2018 at 17:13, Jonathan Wakely wrote:
>
> Ensure we don't try to instantiate __is_constructible_from,
> because there are two partial specializations that are equally good
> matches.
>
> PR libstdc++/80762
> * include/bits/fs_path.h (path::_Path): Use remove_cv_t and
101 - 132 of 132 matches
Mail list logo
