On 07/30/2017 11:55 AM, Thomas Koenig wrote: > Hello world, > > the attached patch performs a runtime-check for the dim arguments > for transformational intrinsic functions. > > I debated with myself a bit if I should make this depend on > bounds checking, but I finally decided against it. The tests > performed are extemely cheap, the values are still in registers, > so any runtime overhead should be small. By contrast, loading > something from an external structure is probably more expensive > (but I didn't bother actually check). > > While visiting the code, I also took the opportunity to make > the rank and dim variables in the 'm' class of functions to index_type, > to make their use consistent. > > Regarding the test case names, I was unable to resist. > > Regression-tested. OK for trunk? >
Looks OK, thanks, Jerry
