[RFC] use symbol_ref flags for rs6000
David Edelsohn
dje@watson.ibm.com
Sat Apr 26 17:35:00 GMT 2003
Richard's AIX xcoff suggestions look correct.
Your patch changes current_file_function_operand(), which is the
only place that tests SYMBOL_REF_FLAG. If the flag no longer is tested
and something is wrong for AIX, things will break whether or not the xcoff
code setting the flag remains.
The test for SYMBOL_REF_LOCAL_P in current_file_function_operand()
should be sufficient.
Richard> The predicate that they are checking is
Richard> SYMBOL_REF_LOCAL_P (x) && SYMBOL_REF_FUNCTION_P (x)
Richard> And given that this is used only in calls, you may or may not
Richard> need to check SYMBOL_REF_FUNCTION_P at all.
You might want to test SYMBOL_REF_FUNCTION_P (x) in
current_file_function_operand() bracketed by ENABLE_CHECKING. It
essentially is an assertion.
David
More information about the Gcc-patches
mailing list