[java] Fix builtins decls

Jan Hubicka jh@suse.cz
Tue May 24 10:05:00 GMT 2005


> On Mon, May 23, 2005 at 07:24:00PM +0100, Andrew Haley wrote:
> > Jan Hubicka writes:
> >  > Hi,
> >  > tree-profiling branch dies horribly by trying to remove
> >  > __builtin_predict calls since these are believed to be trapping in Java.
> >  > I've bootstrapped/regtested i686--linux the attached patch on
> >  > tree-profiling and I am testing it on mainline now too.
> >  > OK (once mainline unslush, as this does not counts as a regression
> >  > there)
> >  > 
> >  > 2005-05-22  Jan Hubicka  <jh@suse.cz>
> >  > 	* builtins.c (define_builtin): Accept new flags parameter.
> >  > 	(initialize_builtins): Mark the builtins const and nothrow accordingly.
> > 
> > That's fine except for __builtin_expect, which is neither const nor
> > nothrow.
> 
> Why wouldn't it be?  It doesn't do anything.  What the arguments do is
> not the builtin's problem.
Ehm...My problem is that builtin_expect itself is marked as throwing and
non-constant.  Tree-profiling branch preplace the builtin_expect calls
by it's argument getting dead EH edges.
Rather then removing the EH edges it seems better to mark the function
as non-trapping and constant...

Honza
> 
> -- 
> Daniel Jacobowitz
> CodeSourcery, LLC



More information about the Java-patches mailing list