On 4/21/21 9:50 PM, Iain Sandoe wrote: > Hi, > > @Martin Hello.
Sorry for the troubles I caused. > I am not sure from the commit messages for this series whether my change > below is complete (it is enough to make bootstrap succeed so I have > applied it anyway). > > you mention TARGET_* but that is too general - however i386/darwin.h does > have some other ISA-specific TARGET_* settings (mostly from well before > my time) so I cannot be 100% sure if more should be removed/changed. > > If you could take a quick look it would be appreciated - I don’t have much > state > for the patch you applied. The patch looks correct to me. Thanks, Martin > > thanks > Iain > > ===== > > The changes from r12-36-g1751bec027f030515889fcf4baa9c91501aafc85 > did not remove the uses of TARGET_ISA_* from i386/darwin.h. > > Fixed thus. > > gcc/ChangeLog: > > * config/i386/darwin.h (TARGET_64BIT): Remove definition > based on TARGET_ISA_64BIT. > (TARGET_64BIT_P): Remove definition based on > TARGET_ISA_64BIT_P(). > --- > gcc/config/i386/darwin.h | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/gcc/config/i386/darwin.h b/gcc/config/i386/darwin.h > index c81db9bf09f..2657dfe266e 100644 > --- a/gcc/config/i386/darwin.h > +++ b/gcc/config/i386/darwin.h > @@ -25,11 +25,6 @@ along with GCC; see the file COPYING3. If not see > #undef DARWIN_X86 > #define DARWIN_X86 1 > > -#undef TARGET_64BIT > -#define TARGET_64BIT TARGET_ISA_64BIT > -#undef TARGET_64BIT_P > -#define TARGET_64BIT_P(x) TARGET_ISA_64BIT_P(x) > - > #ifdef IN_LIBGCC2 > #undef TARGET_64BIT > #ifdef __x86_64__
