[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