This looks like a relatively easy bug to fix, but a hard one for me to test without an example. Can you
propose a patch please?

Thanks,

-Chris

There's two ways. (I think I'd prefer (a).)

(a) Check for TargetInstrInfo explicitly

--- DAGISelEmitter.cpp 8 Nov 2006 23:01:03 -0000 1.281
+++ DAGISelEmitter.cpp 14 Nov 2006 03:11:13 -0000
@@ -3387,7 +3387,13 @@
}

void DAGISelEmitter::EmitInstructionSelector(std::ostream &OS) {

• std::string InstNS = Target.inst_begin()->second.Namespace;

• std::string InstNS;

• CodeGenTarget::inst_iterator i;

• for (i = Target.inst_begin(); i != Target.inst_end(); ++i) {

• InstNS = i->second.Namespace;

• if (InstNS != "TargetInstrInfo")

•   break;
  

• }
  if (!InstNS.empty()) InstNS += "::";

  // Group the patterns by their top-level opcodes.

(b) Check for INLINEASM & PHI explicitly:

diff -u -r1.281 DAGISelEmitter.cpp
--- DAGISelEmitter.cpp 8 Nov 2006 23:01:03 -0000 1.281
+++ DAGISelEmitter.cpp 14 Nov 2006 03:16:36 -0000
@@ -3387,7 +3387,15 @@
}

void DAGISelEmitter::EmitInstructionSelector(std::ostream &OS) {

• std::string InstNS = Target.inst_begin()->second.Namespace;

• std::string InstNS;

• CodeGenTarget::inst_iterator i;

• for (i = Target.inst_begin(); i != Target.inst_end(); ++i) {

• std::string name = i->second.TheDef->getName();

• if (name != "INLINEASM" && name != "PHI") {

•   InstNS = i->second.Namespace;
  

•   break;
  

• }

• }
  if (!InstNS.empty()) InstNS += "::";

  // Group the patterns by their top-level opcodes.

Thanks,

Option #​1 sounds good to me! Thanks, I applied your patch:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20061120/040138.html

Thanks,

-Chris