This patch attempts to improve the diagnostic for generic matches, if a
dummy procedure is used which has nonmatching characteristics.
Before the patch:
call gen(sub)
1
Error: There is no specific subroutine for the generic 'gen' at (1)
After the patch:
call gen(sub
Dear Tobias,
Failures in 32 bit mode again!-(
With your patch, I get the following failures with -m32:
FAIL: gfortran.dg/class_48.f90 -O2 (test for excess errors)
FAIL: gfortran.dg/class_48.f90 -Os (test for excess errors)
FAIL: gfortran.dg/finalize_18.f90 -O scan-tree-dump-times original
On 18 June 2013 17:41, Ramana Radhakrishnan wrote:
> On 06/18/13 09:50, Zhenqiang Chen wrote:
>>
>> Hi,
>>
>> During expand, function vcond inverses some CMP, e.g.
>>
>> a LE b -> b GE a
>>
>> But if "b" is "CONST0_RTX", "b GE a" will be an illegal insn.
>>
>> (insn 933 932 934 113 (set (reg:V
On 18/10/12 18:48, Ramana Radhakrishnan wrote:
> +(define_insn_reservation "cortex_a15_vfp_adds_subs" 6
> + (and (eq_attr "tune" "cortexa15")
> + (eq_attr "type" "fadds"))
> + "ca15_issue1,ca15_cx_vfp")
Ramana,
I noticed you use here the fadds value for the type attribute to activate this
On 06/25/13 13:42, Iyer, Balaji V wrote:
What remaining obstacles are there to sharing most of the expansion
code between C and C++? That can be a separate patch, of course.
Jason
??
I changed the method by which class (and alias) template constraints
are checked. Rather than waiting until instantiation, they are now
checked after computing the binding in lookup_class template.
Actually, after the hashtable lookup so we don't re-check constraints
for previously instantiated spe
The patch changes the out-of-bounds message for "k==11"
z(i)%y(j)%x(k)=0
from:
Fortran runtime error: Index '11' of dimension 1 of array 'z' above
upper bound of 10
to
Fortran runtime error: Index '11' of dimension 1 of array 'z%y%x'
above upper bound of 10
(For j out of bounds, it woul
This is a small cleanup to the Cilk Plus mention in our documentation,
but more importantly, it clarifies that the Cilk Plus implementation in
GCC is only partial.
OK for trunk?
* doc/invoke.texi (-fcilkplus): Clarify that implementation is
incomplete.
diff --git a/gcc/doc/invo
On 06/25/2013 08:50 AM, Jason Merrill wrote:
I had missed a few files in my patch anyway (I was doing too much at once).
On 06/25/2013 08:27 AM, Ed Smith-Rowland wrote:
+ else if (token->type == CPP_KEYWORD)
+{
+ error ("unexpected keyword;"
+ " Remove space between quote
Ping http://gcc.gnu.org/ml/gcc-patches/2013-06/msg00475.html
On Sun, 9 Jun 2013, Marc Glisse wrote:
Hello,
this patch documents that __builtin_isinf_sign returns +-1 for +-Inf. This
builtin was created so it could be used by a libc that has a stronger
guarantee than the standard, and for gli
Ping http://gcc.gnu.org/ml/gcc-patches/2013-06/msg00470.html
On Sun, 9 Jun 2013, Marc Glisse wrote:
Hello,
when porting __builtin_shuffle from C to C++, I ignored all the C++
specificities and added some extra bugs. This should improve things a bit.
Bootstrap+testsuite on x86_64-linux-gnu.
Ping http://gcc.gnu.org/ml/gcc-patches/2013-06/msg00478.html
On Sun, 9 Jun 2013, Marc Glisse wrote:
Hello,
this patch removes 2 builtins that are undocumented, unused, and have
confusing semantics. Bootstrap+testsuite on x86_64-linux-gnu.
2013-06-10 Marc Glisse
PR target/57224
This is a followup for the C++ changes (your pt.c suggested changes, etc).
Only a few minor changes are needed, since c_finish_cilk_simd_loop() is
shared between C/C++ and it creates the appropriate CILK_SIMD tree node.
Is this what you had in mind?
commit db2127098137dea6c246041e0d763a57a174f
Ping (re-attaching the patch)
2013-06-26 Marc Glisse
PR other/57324
* hwint.h (HOST_WIDE_INT_UC, HOST_WIDE_INT_1U, HOST_WIDE_INT_M1,
HOST_WIDE_INT_M1U): New macros.
* fold-const.c (sign_bit_p, build_range_check, fold_unary_loc,
fold_binary_loc, fold_ter
Hi Tobias,
The patch changes the out-of-bounds message for "k==11"
z(i)%y(j)%x(k)=0
from:
Fortran runtime error: Index '11' of dimension 1 of array 'z' above
upper bound of 10
to
Fortran runtime error: Index '11' of dimension 1 of array 'z%y%x'
above upper bound of 10
(For j out of bou
On Wed, Jun 26, 2013 at 09:44:03AM -0500, Aldy Hernandez wrote:
> This is a followup for the C++ changes (your pt.c suggested changes, etc).
>
> Only a few minor changes are needed, since c_finish_cilk_simd_loop()
> is shared between C/C++ and it creates the appropriate CILK_SIMD
> tree node.
>
>
The Go language does not permit global variables named init. That name
is reserved for functions that run when the program starts. This patch,
from Rémy Oudompheng, implements that restriction in gccgo.
Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu.
Committed to mainline and 4.8 b
Hi,
On Tue, Jun 25, 2013 at 03:14:47PM +0200, Jan Hubicka wrote:
> > the patch reportedly fixes an issue when LTO building chromium. It
> > adds references that are created by IPA-CP when propagating references
> > from aggregates (the scalar case is handled by the call graph
> > infrastructure).
Ping.
On 06/12/2013 11:46 AM, Meador Inge wrote:
> Hi All,
>
> This patch adds the needed 'dg-require-effective-target sync_*' lines to some
> of the atomic tests so that they will not be run if the appropriate atomic
> support is not available.
>
> OK for trunk?
>
> 2013-06-12 Meador Inge
>
On 06/26/13 10:33, Jakub Jelinek wrote:
On Wed, Jun 26, 2013 at 09:44:03AM -0500, Aldy Hernandez wrote:
This is a followup for the C++ changes (your pt.c suggested changes, etc).
Only a few minor changes are needed, since c_finish_cilk_simd_loop()
is shared between C/C++ and it creates the appr
Hi!
On the gomp4 branch I've noticed a miscompilation of the simd-3.C
testcase I'm adding there, but even say
int a[1024] __attribute__((aligned (32))) = { 1 };
int b[1024] __attribute__((aligned (32))) = { 1 };
unsigned short c[1024] __attribute__((aligned (32))) = { 1 };
__attribute__((noinline
On 06/26/2013 01:31 PM, Iyer, Balaji V wrote:
Attached, please find a fixed patch and ChangeLog entries:
This patch seems to be missing some hunks that are described in the
ChangeLog and were present in the previous patch, such as
* cp-array-notation.c (cp_length_mismatch_in
On 06/09/2013 07:09 AM, Marc Glisse wrote:
+ arg0 = build_non_dependent_expr (arg0);
+ arg1 = build_non_dependent_expr (arg1);
+ arg2 = build_non_dependent_expr (arg2);
+}
+ return c_build_vec_perm_expr (loc, arg0, arg1, arg2, complain & tf_error);
This is wrong; the places
Tobias Burnus wrote:
With coarrays, allocation/deallocation of coarrays requires a
synchronization with all other images. Thus, the standard restricts
changing the allocation status to: ALLOCATE and DEALLOCATE statements
plus end-of-scope deallocation.
In particular, with intrinsic assignment
On 06/26/2013 09:43 AM, Ed Smith-Rowland wrote:
+ if (bad_encoding_prefix)
+ error ("invalid encoding prefix in literal operator");
+ {
+ tree string_tree = USERDEF_LITERAL_VALUE (token->u.value);
No need to open a nested block for a declaration now that we're
compiling a
Hi!
On Sat, 22 Jun 2013 08:15:46 -0700, Ian Lance Taylor wrote:
> Go can work without split stack. In that case libgo will use much
> larger stacks for goroutines, to reduce the chance of running out of
> stack space (see StackMin in libgo/runtime/proc.c). So the number of
> simultaneous gorout
Hi!
On Fri, 14 Jun 2013 16:42:06 +0200, I wrote:
> I have now pushed my working branch to tschwinge/t/hurd/go, and will
> update this from time to time, both for integrating further changes (my
> own as well as those that you send me), and merge in GCC trunk changes.
> On that branch, use somethin
Hello,
this patch tries to implement DR1164. It passes bootstrap+testsuite on
x86_64-unknown-linux-gnu.
2013-06-27 Marc Glisse
PR c++/57172
gcc/cp/
* pt.c (more_specialized_fn): If both arguments are references,
give priority to an lvalue.
gcc/testsuite/
*
Hi,
A set of five patches will be sent shortly as the gcc part of changes
that add support for ILP32 in the AArch64 baremetal toolchain.
The five patches will be organized as the following:
1. Configury changes;
2. AArch64 backend changes that add necessary instruction patterns and
update t
This patch adds the configuration changes to the AArch64 GCC to support:
* -milp32 and -mlp64 options in the compiler and the driver
* multilib of ilp32 and/or lp64 libraries
* differentiation of basic types in the compiler backend
The patch enables --with-multilib-list configuration option for
This patch updates the AArch64 backend to support the small absolute and
small PIC addressing models for ILP32; it also updates a number of other
backend macros and hooks in order to support ILP32.
OK for the trunk?
Thanks,
Yufeng
gcc/
* config/aarch64/aarch64.c (POINTER_BYTES): New
This patch updates assign_parm_find_data_types to assign passed_mode and
nominal_mode with the mode of the built pointer type instead of the
hard-coded Pmode in the case of pass-by-reference. This is in line with
the assignment to passed_mode and nominal_mode in other cases inside the
function
The attached patch fixes a few gcc test cases.
Thanks,
Yufeng
gcc/testsuite/
* gcc.dg/20020219-1.c: Skip the test on aarch64*-*-* in ilp32.
* gcc.target/aarch64/aapcs64/test_18.c (struct y): Change the field
type from long to long long.
* gcc.target/aarch64/ato
In Go 1 division by an integer constant 0 was changed from a runtime
error to a compile time error. This patch from Rémy Oudompheng
implements that in gccgo. This required updating a couple of tests to
new copies from the master sources. Bootstrapped and ran Go testsuite
on x86_64-unknown-linux-
This patch defines _ILP32 and __ILP32__ for the AArch64 port when the
ILP32 ABI is in use.
This helps libraries, e.g. libgloss and glibc, recognize which model is
being compiled.
OK for the trunk?
Thanks,
Yufeng
gcc/
* config/aarch64/aarch64.h (TARGET_CPU_CPP_BUILTINS): Define _ILP
In Go 1 the fallthrough statement is no longer permitted in the last
case of a switch, as there is no code to which to fall through. This
patch from Rémy Oudompheng implements this restriction in gccgo.
Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu.
Committed to mainline and 4.8 br
On Wed, Jun 26, 2013 at 3:33 PM, Yufeng Zhang wrote:
> This patch adds the configuration changes to the AArch64 GCC to support:
>
> * -milp32 and -mlp64 options in the compiler and the driver
> * multilib of ilp32 and/or lp64 libraries
> * differentiation of basic types in the compiler backend
>
>
The warn_type should also be guarded with float type check:
warn_type = (warn_real_conversion
&& (FLOAT_TYPE_P (type) || FLOAT_TYPE_P (expr_type)))
? OPT_Wreal_conversion
: OPT_Wconversion;
Also why did you put the warn_type code inside the default?
David
On Mon, Jun 24, 2013 at 10:03 PM, Sha
On Wed, Jun 26, 2013 at 3:39 PM, Yufeng Zhang wrote:
> This patch updates assign_parm_find_data_types to assign passed_mode and
> nominal_mode with the mode of the built pointer type instead of the
> hard-coded Pmode in the case of pass-by-reference. This is in line with the
> assignment to passe
On 06/27/13 00:04, Andrew Pinski wrote:
On Wed, Jun 26, 2013 at 3:39 PM, Yufeng Zhang wrote:
This patch updates assign_parm_find_data_types to assign passed_mode and
nominal_mode with the mode of the built pointer type instead of the
hard-coded Pmode in the case of pass-by-reference. This is i
On Wed, Jun 26, 2013 at 4:41 PM, Yufeng Zhang wrote:
> On 06/27/13 00:04, Andrew Pinski wrote:
>>
>> On Wed, Jun 26, 2013 at 3:39 PM, Yufeng Zhang
>> wrote:
>>>
>>> This patch updates assign_parm_find_data_types to assign passed_mode and
>>> nominal_mode with the mode of the built pointer type ins
On Wed, Jun 26, 2013 at 4:51 PM, Andrew Pinski wrote:
> On Wed, Jun 26, 2013 at 4:41 PM, Yufeng Zhang wrote:
>> On 06/27/13 00:04, Andrew Pinski wrote:
>>>
>>> On Wed, Jun 26, 2013 at 3:39 PM, Yufeng Zhang
>>> wrote:
This patch updates assign_parm_find_data_types to assign passed_mode a
Sorry, my port was bad. I am going to revert this patch and redo it.
Thanks,
Sharad
On Wed, Jun 26, 2013 at 4:03 PM, Xinliang David Li wrote:
> The warn_type should also be guarded with float type check:
>
> warn_type = (warn_real_conversion
> && (FLOAT_TYPE_P (type) || FLOAT_TYPE_P (expr_type)
On 06/27/13 00:51, Andrew Pinski wrote:
On Wed, Jun 26, 2013 at 4:41 PM, Yufeng Zhang wrote:
On 06/27/13 00:04, Andrew Pinski wrote:
On Wed, Jun 26, 2013 at 3:39 PM, Yufeng Zhang
wrote:
This patch updates assign_parm_find_data_types to assign passed_mode and
nominal_mode with the mode of th
On 06/27/13 00:57, Andrew Pinski wrote:
On Wed, Jun 26, 2013 at 4:51 PM, Andrew Pinski wrote:
On Wed, Jun 26, 2013 at 4:41 PM, Yufeng Zhang wrote:
On 06/27/13 00:04, Andrew Pinski wrote:
On Wed, Jun 26, 2013 at 3:39 PM, Yufeng Zhang
wrote:
This patch updates assign_parm_find_data_types to
On Wed, 26 Jun 2013, Yufeng Zhang wrote:
> This patch defines _ILP32 and __ILP32__ for the AArch64 port when the ILP32
> ABI is in use.
>
> This helps libraries, e.g. libgloss and glibc, recognize which model is being
> compiled.
GCC already defines _LP64 and __LP64__ in architecture-independent
OK.
Jason
We already added a helpful error message for this case, but we should
also avoid crashing.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit b87fed16be1aad58770349909ab8045088ed28ff
Author: Jason Merrill
Date: Fri Jun 21 08:28:49 2013 -0400
PR c++/57408
* semantics.c (add_captu
A question from a student of a committee member led me to notice that we
were handling overflow in a class model of Fibonacci numbers oddly:
C++11 says that arithmetic overflow causes an expression to not be
constant, but instead of enforcing that on the actual affected variable
initialization,
Just something else I saw while analyzing dumps from an unrelated set of
changes.
It's relatively common to see sequences like this:
# parent_1 = PHI
_11 = single_tree_10(D) != 0;
_12 = parent_1 == 0B;
_13 = _11 & _12;
if (_13 != 0)
goto ;
else
goto ;
Where VRP can deduc
I reverted the earlier broken patch. I am including an updated patch
which warns only for real conversion, not for integral conversions. I
also updated the test case to include an integral conversion (int to
char) which doesn't emit the warning with the -Wreal-conversion flag.
Bootstrapped and tes
Ok.
David
On Wed, Jun 26, 2013 at 10:16 PM, Sharad Singhai wrote:
> I reverted the earlier broken patch. I am including an updated patch
> which warns only for real conversion, not for integral conversions. I
> also updated the test case to include an integral conversion (int to
> char) which do
52 matches
Mail list logo
