>
> Does it make sense to make this work?  It seems much better than
> having to hack arm.h for every subtarget that doesn't use @ as a
> comment character.
>

I'm afraid I didn't understand your point. I think the problem is
- Target can have inline asm extensions (not only comments - ARM
target has two extensions: '%@' and '%r').
So we need a way to define this extensions. To do this, I copied the
idea used by the gcc codegen.

arm.h code for gcc codegen:

/* These are a couple of extensions to the formats accepted
   by asm_fprintf:
     %@ prints out ASM_COMMENT_START
     %r prints out REGISTER_PREFIX reg_names[arg]  */
#define ASM_FPRINTF_EXTENSIONS(FILE, ARGS, P)           \
  case '@':                                             \
    fputs (ASM_COMMENT_START, FILE);                    \
    break;                                              \
                                                        \
  case 'r':                                             \
    fputs (REGISTER_PREFIX, FILE);                      \
    fputs (reg_names [va_arg (ARGS, int)], FILE);       \
    break;


Lauro
_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Reply via email to