[PATCH] Fixed the Intel-syntax X86 disassembler to respect the option for hexadecimal immediates.

Daniel Malea daniel.malea at intel.com
Thu Aug 1 14:04:50 PDT 2013


  Ah, sorry for the last-minute nit, but after building with clang (trunk), I'm seeing some new warnings:


  ../lib/MC/MCInstPrinter.cpp:74:3: warning: default label in switch which covers all enumeration values [-Wcovered-switch-default]
    default: llvm_unreachable("Invalid");
    ^
  ../lib/MC/MCInstPrinter.cpp:97:3: warning: default label in switch which covers all enumeration values [-Wcovered-switch-default]
    default: llvm_unreachable("Invalid");
    ^


  I don't believe the default case is needed at all.

http://llvm-reviews.chandlerc.com/D1243

BRANCH
  default

ARCANIST PROJECT
  llvm



More information about the llvm-commits mailing list