[llvm-commits] [llvm] r159203 - in /llvm/trunk: lib/Target/ARM/ARMAsmPrinter.cpp lib/Target/CellSPU/SPUAsmPrinter.cpp lib/Target/Hexagon/HexagonAsmPrinter.cpp lib/Target/MBlaze/MBlazeAsmPrinter.cpp lib/Target/NVPTX/NVPTXAsmPrinter.cpp lib/Target/Po...

[Rafael Espíndola]

On 15 July 2013 14:52, Jack Carter <Jack.Carter at imgtec.com> wrote:
> Aaron,
>
> Thanks for catching the omission of the call to the generic AsmPrinter::PrintAsmOperand(). Better late than never.
>
> What I don't understand is why a switch statement with only the default filled bad? It is a template for future additions and any compiler worth it's salt will compile it away.

We normally don't do that. Keep the code as simple as possible for
what it currently implements.

> Jack

Cheers,
Rafael