2008/8/14 Robert Dewar <[EMAIL PROTECTED]>: > Manuel López-Ibáñez wrote: > >> My proposal is to enable this while not in release mode (like we >> enable checks and dump statistics and output from the optimizers), so >> developers notice wrong locations and open a PR and maybe even fix >> them. Then maybe let users or distributions configure the default as >> they wish, with our releases defaulting to off. > > Won't this have a negative effect on the test infrastructure? It > sure would for our Ada testing, where we have hundreds of thousands > of diagnostic messages in our test suites.
Even when enabled with -fdiagnostics-show-caret and configured with --enable-languages=all,ada,obj-c++, all acats and gnat tests pass. Of course, we should disable it with -fno-diagnostics-show-caret as we currently do for -fshow-colum in lib/gcc.exp. You'll notice that is what my patch does if you read it. Cheers, Manuel.
