This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Patch, Fortran] Check arguments of elemental intrinsic subroutines for conformance


Paul Richard Thomas wrote:
Is it ok if no regressions occur?  What do you think, should I commit now
and work then on the "real" part of PR 35681 or should I try to do both in a
single larger patch?  I'm fine with both options, although I don't expect
that there's a significant "dependance" of both problems, so seperate
patches seem totally fine.

I think that this is good to be committed now. It is close on obvious and must be done, regardless of the fix for PR35681.

OK for trunk, subject to regtest.

No regressions (here on GNU/Linux-x86-32, thanks Dominique for your testing, too!), committed to trunk as rev. 141175. Thanks for the quick review!


PS As a matter of curiosity, how will you fix PR35681?  It clearly
needs a temporary, so how will you trigger it?

I have to admit that I've not yet looked into it really deeply; I think I will first start some playing around and try to find out where the need for a temporary is missed... Well, simply I don't know yet :) But hopefully I'll find a way ;)


Yours,
Daniel

--
Done:  Arc-Bar-Cav-Rog-Sam-Val-Wiz
To go: Hea-Kni-Mon-Pri-Ran-Tou


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]