[llvm-commits] [PATCH, PING] Peephole Infrastructure improvements and (ARM, T, T2) TSTrr optimizations
wendling at apple.com
Thu Oct 28 16:43:52 PDT 2010
Sorry about the long response time. A few general comments:
• Watch out for consistent spacing.
• You should use "llvm::next" instead of "next". The latter conflicts with an "std" name (if I recall correctly).
• You call 'new' on the Opportunity classes in a situation where it expects to return a "bool" value. What's going on here?
On Oct 25, 2010, at 1:28 AM, Gabor Greif wrote:
> Attached the diff against recent trunk.
> On 10/21/10, Gabor Greif <ggreif at gmail.com> wrote:
>> Hi all,
>> the last weeks I've been working on a flexible infrastructure for
>> peephole optimizations, which is potentially target independent and
>> extensible without needing interface changes.
>> The result of my work is attached. It moves all current ARM peepholes
>> over to the new architecture and adds TSTrr-related optimizations too.
>> The ordering and forward referencing of functions is still suboptimal,
>> but this is only done to keep the patch size manageable. I plan to
>> reorder in a cleanup commit after this patch has landed. Also some
>> currently freestanding functions will become methods.
>> You can also see the code in its entirety here:
>> Feedback is welcome.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits