[PATCH] Implement the ldr-pseudo opcode for ARM assembly
    Amara Emerson 
    amara.emerson at arm.com
       
    Wed Dec  4 08:32:37 PST 2013
    
    
  
Same here.
Jim, is this ok to go in?
Amara
-----Original Message-----
From: llvm-commits-bounces at cs.uiuc.edu
[mailto:llvm-commits-bounces at cs.uiuc.edu] On Behalf Of Tim Northover
Sent: 04 December 2013 16:16
To: David Peixotto
Cc: Commit Messages and Patches for LLVM
Subject: Re: [PATCH] Implement the ldr-pseudo opcode for ARM assembly
> Still waiting for internal approval for phabricator. Would it be ok to
> conduct the review the old-fashioned way for now?
I think it looks reasonable too. I've played around with it and I
can't see anything wrong with the Darwin output.
One small thing I (well, clang) noticed is that the variable
ConstantPoolCounter is unused.
Cheers.
Tim.
_______________________________________________
llvm-commits mailing list
llvm-commits at cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
    
    
More information about the llvm-commits
mailing list