[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