Re: [RFA][PATCH] Minor erroneous path isolation followups

2013-11-06 Thread Jakub Jelinek
On Wed, Nov 06, 2013 at 07:48:49PM -0700, Jeff Law wrote: > * doc/invoke.texi (-fisolate-erroneous-paths): Document. > > * gimple-ssa-isolate-paths.c (gate_isolate_erroneous_paths): > No longer check if we have __builtin_trap, assume it's > available. > > java/ > > * builtins

Re: [RFA][PATCH] Minor erroneous path isolation followups

2013-11-06 Thread Jeff Law
On 11/06/13 00:12, Jakub Jelinek wrote: On Tue, Nov 05, 2013 at 11:15:30PM -0700, Jeff Law wrote: * java/builtins.c (initialize_builtins): Provide __builtin_trap. Without java/ ;) Yes. I was aware of that. I probably should have done java/ * builtins.c ... --- a/gcc/ja

Re: [RFA][PATCH] Minor erroneous path isolation followups

2013-11-05 Thread Jakub Jelinek
On Tue, Nov 05, 2013 at 11:15:30PM -0700, Jeff Law wrote: > * java/builtins.c (initialize_builtins): Provide __builtin_trap. Without java/ ;) > --- a/gcc/java/builtins.c > +++ b/gcc/java/builtins.c > @@ -580,6 +580,9 @@ initialize_builtins (void) > build_function_type_list (

[RFA][PATCH] Minor erroneous path isolation followups

2013-11-05 Thread Jeff Law
Per a discussion between Richi and myself, this patch adds a __builtin_trap to the Java front end. If someone could look at that code closely to ensure I didn't muck it up, I'd appreciated it. With that code in the Java front-end, we can remove the hack from gate_isolate_erroneous_paths.