> Okay, if you update from CVS, this problem should be fixed, at least for > the AsmPrinter. Please try it out (without AAA), and let me know if you > have problems with any other part of the code generator.The behaviour now is: The TargetInstrInfo namespace is used for the PHI instruction regardless of the presence of the AAA instruction. In ARMGenInstrNames.inc, the TargetInstrInfo namespace is used if the AAA instruction isn't defined and the ARM namespace is used if it is defined.> Thanks! > > -ChrisBest Regards, Rafael
> > In ARMGenInstrNames.inc, the TargetInstrInfo namespace is used if the > > AAA instruction isn't defined and the ARM namespace is used if it is > > defined. > > I don't follow here. XXXGenInstrNames.inc shouldn't have namespaces at > all. Can you paste a few lines out of the file that you are seeing?Sure: with AAA defined, ARMGenInstrNames.inc looks like: ----------------------------------------- namespace llvm { namespace ARM { . . . } } -------------------------------------- and without AAA, ARMGenInstrNames.inc looks like: -------------------------------------------- namespace llvm { namespace TargetInstrInfo { . . . } } --------------------------------------------> Thanks, > > -ChrisThanks, Rafael
On Mon, 1 May 2006, [UTF-8] Rafael Esp?ndola wrote:>> Okay, if you update from CVS, this problem should be fixed, at least for >> the AsmPrinter. Please try it out (without AAA), and let me know if you >> have problems with any other part of the code generator. > The behaviour now is: > The TargetInstrInfo namespace is used for the PHI instruction > regardless of the presence of the AAA instruction.Good, this is expected. TargetInstrInfo::PHI == ARM::PHI.> In ARMGenInstrNames.inc, the TargetInstrInfo namespace is used if the > AAA instruction isn't defined and the ARM namespace is used if it is > defined.I don't follow here. XXXGenInstrNames.inc shouldn't have namespaces at all. Can you paste a few lines out of the file that you are seeing? Thanks, -Chris -- http://nondot.org/sabre/ http://llvm.org/