[LLVMdev] Typos in ARMInstrInfo.td ?

George Russell george at codeplay.com
Fri Dec 16 01:50:12 PST 2011


Hi,

I think there are a set of typos in the ATOMIC_LOAD_UMIN_I* and 
ATOMIC_LOAD_UMAX_I*
pseudo-instructions .

Specifically,

def ATOMIC_LOAD_MIN_I32 : PseudoInst<
                 (outs GPR:$dst), (ins GPR:$ptr, GPR:$val), NoItinerary,
                 [(set GPR:$dst, (atomic_load_min_32 GPR:$ptr, GPR:$val))]>;

and

def ATOMIC_LOAD_UMIN_I32 : PseudoInst<
                 (outs GPR:$dst), (ins GPR:$ptr, GPR:$val), NoItinerary,
                 [(set GPR:$dst, (atomic_load_min_32 GPR:$ptr, GPR:$val))]>;

both reference atomic_load_min_32, and the latter should instead 
reference atomic_load_umin_32

Added here,
See 
http://llvm.org/viewvc/llvm-project/llvm/tags/RELEASE_30/final/lib/Target/ARM/ARMInstrInfo.td?diff_format=h&view=diff&r1=130233&r2=130234

Still present in trunk
http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMInstrInfo.td?diff_format=h&view=annotate

Cheers,
George



More information about the llvm-dev mailing list