[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