[PATCH] [AArch64] Add pass to enable additional comparison optimizations by CSE
Chad Rosier
mcrosier at codeaurora.org
Wed Aug 27 08:21:51 PDT 2014
>>! In D5044#11, @sdmitrouk wrote:
> Chad,
>
>> Is this strictly target-specific?
>
> I guess not, other targets can benefit of similar optimizations as well.
Rather, is the implementation necessarily target-specific? I imagine other targets would benefit, but I'm asking if we could have a more generic solution.
> Actually, not that much code to share among
> different targets. Is it enough to create target-independent pass? If
> yes, I'll file a bug.
Based on your response, my impression is that this requires a mostly target-specific implementation. I can't provide an educated argument for or against your approach, so I think the purpose of the bug would be to continue the discussion. I'll leave it to your good judgment to decide if this discussion is really necessary. In the meantime, I have no issue with the current solution.
Chad
http://reviews.llvm.org/D5044
More information about the llvm-commits
mailing list