> Also, you have removed an optimization that calls the simpler string_select > routine when stride == 1. It would be OK with me if string_strided_select > included that optimization, but I think it's important.
I haven't reviewed this patch in detail, but based on this comment wanted to note that if the conditional also checked whether the range was stridable, that would convert a runtime check into a compile-time one and permit the conditional to be folded in the (common) case of using non-stridable ranges (ones in which the stride is always 1 as specified through their type at compile-time). -Brad ------------------------------------------------------------------------------ Rapidly troubleshoot problems before they affect your business. Most IT organizations don't have a clear picture of how application performance affects their revenue. With AppDynamics, you get 100% visibility into your Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! http://pubads.g.doubleclick.net/gampad/clk?id=84349351&iu=/4140/ostg.clktrk _______________________________________________ Chapel-developers mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/chapel-developers
