]> granicus.if.org Git - llvm/commitdiff
[AVR] Remove a few very old TODOs that don't have enough context to understand
authorDylan McKay <me@dylanmckay.io>
Tue, 11 Jul 2017 05:14:40 +0000 (05:14 +0000)
committerDylan McKay <me@dylanmckay.io>
Tue, 11 Jul 2017 05:14:40 +0000 (05:14 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@307622 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/AVR/AVRAsmPrinter.cpp
lib/Target/AVR/AVRRegisterInfo.td

index f0c7b11895b4a08f0129dca9f7ed5330bd94c281..c058c9e1f5348485c0eb3add24077b6369328de9 100644 (file)
@@ -149,7 +149,10 @@ bool AVRAsmPrinter::PrintAsmMemoryOperand(const MachineInstr *MI,
   (void)MO;
   assert(MO.isReg() && "Unexpected inline asm memory operand");
 
-  // TODO: We can look up the alternative name for the register if it's given.
+  // TODO: We should be able to look up the alternative name for
+  // the register if it's given.
+  // TableGen doesn't expose a way of getting retrieving names
+  // for registers.
   if (MI->getOperand(OpNum).getReg() == AVR::R31R30) {
     O << "Z";
   } else {
index 8d663637a2b11cb5bfef96ef96f6c991d830adb4..8162f12052be5aa22bf962d8368152ae92de28c3 100644 (file)
@@ -202,8 +202,6 @@ def ZREG : RegisterClass<"AVR", [i16], 8, (add R31R30)>;
 // Register class used for the stack read pseudo instruction.
 def GPRSP: RegisterClass<"AVR", [i16], 8, (add SP)>;
 
-//:TODO: if we remove this we get an error in tablegen
-//:TODO: this is just a hack, remove it once add16 works!
 // Status register.
 def SREG : AVRReg<14, "FLAGS">, DwarfRegNum<[88]>;
 def CCR : RegisterClass<"AVR", [i8], 8, (add SREG)>