Hi! On Fri, 2 Mar 2018 12:47:23 -0800, Cesar Philippidis <ce...@codesourcery.com> wrote: > The attached patch is the last one in the vector length extension > series. It consists of some tweaks to the libgomp nvptx plugin to > accommodate larger vectors along with two test cases. > > I only added two test cases because there's really not much interesting > going on with longer vector lengths. We should eventually add more tests > cases to handle situations where the nvptx BE falls back to using a > shorter vector length. But right now GCC just makes those changes > silently. There is precedent for the nvptx BE to emit a warning when > vector length != 32, but that might be too verbose. On one hand, it > could be argued that the compiler should error if it cannot satisfy the > user's request. On the other hand, falling back to a smaller vector > length ensures correctness. > > Thomas, do you have any thoughts on the warnings/errors or there lack of?
Yeah, warning when "vector_length([bigger than 32])" is reduced to "vector_length(32)" is probably too verbose/not useful. But, it'd be useful to have in "-fopt-info-omp"? Grüße Thomas
