diff options
| author | Alan Modra <amodra@gmail.com> | 2021-07-10 15:58:44 +0930 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2021-07-14 21:52:20 +0930 |
| commit | 9cce9fab90e0951c6816d97801e124e392e74dd9 (patch) | |
| tree | 30af6ff2fa37c54f9644201b08d17ede2f8fbc47 /gas/config/tc-alpha.h | |
| parent | 51d29b8cacd75865a9ffc46cffffe1e47c33ccff (diff) | |
| download | fsf-binutils-gdb-9cce9fab90e0951c6816d97801e124e392e74dd9.zip fsf-binutils-gdb-9cce9fab90e0951c6816d97801e124e392e74dd9.tar.gz fsf-binutils-gdb-9cce9fab90e0951c6816d97801e124e392e74dd9.tar.bz2 | |
gas: default TC_VALIDATE_FIX_SUB to 0
gas/write.c provides a fallback TC_VALIDATE_FIX_SUB define that can be
a problem for some targets, the problem being that a non-zero
definition of TC_VALIDATE_FIX_SUB says that some uses of fx_subsy are
OK, in effect that the target will handle fx_subsy in md_apply_fix
and/or tc_gen_reloc. A lot of targets don't have the necessary
md_apply_fix and tc_gen_reloc support. So a safer default is to
disallow fx_subsy by default.
I've had a good look over target usage of fx_subsy, and think I've
caught all the cases where targets need TC_VALIDATE_FIX_SUB. Possible
failures would be limited to alpha, microblaze, ppc and s390 (the
targets that define UNDEFINED_DIFFERENCE_OK), or targets that generate
fixups with BFD_RELOC_GPREL32/16 and use a syntax explicitly showing
a difference expression.
* write.c (TC_VALIDATE_FIX_SUB): Default to 0.
* config/tc-hppa.h (TC_VALIDATE_FIX_SUB): Define.
* config/tc-microblaze.h (TC_VALIDATE_FIX_SUB): Define.
* config/tc-alpha.h (TC_VALIDATE_FIX_SUB): Define for ECOFF.
* config/tc-ppc.h (TC_VALIDATE_FIX_SUB): Don't define for ELF.
Do define for XCOFF.
Diffstat (limited to 'gas/config/tc-alpha.h')
| -rw-r--r-- | gas/config/tc-alpha.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/gas/config/tc-alpha.h b/gas/config/tc-alpha.h index b7d7e87..ed0ae63 100644 --- a/gas/config/tc-alpha.h +++ b/gas/config/tc-alpha.h @@ -116,6 +116,11 @@ extern void alpha_handle_align (struct frag *); #ifdef OBJ_ECOFF #define tc_frob_file_before_adjust() alpha_frob_file_before_adjust () extern void alpha_frob_file_before_adjust (void); + +#define TC_VALIDATE_FIX_SUB(FIX, SEG) \ + ((md_register_arithmetic || (SEG) != reg_section) \ + && ((FIX)->fx_r_type == BFD_RELOC_GPREL32 \ + || (FIX)->fx_r_type == BFD_RELOC_GPREL16)) #endif #define DIFF_EXPR_OK /* foo-. gets turned into PC relative relocs. */ |