[PATCH][LoopStrengthReduce] Don't trim a formula that uses a subset of required registers

Adam Nemet anemet at apple.com
Mon Apr 28 16:40:12 PDT 2014


On Apr 28, 2014, at 4:32 PM, Chandler Carruth <chandlerc at google.com> wrote:

> 
> On Mon, Apr 28, 2014 at 4:09 PM, Adam Nemet <anemet at apple.com> wrote:
> I’ll revert the test case until we investigate but clang does get substituted in test/lit.cfg.
> 
> What if you don't have Clang checked out?
> 
> This is just not the right way to test this. We need to have an IR test case, not a C test case.


The problem is that this test case is sensitive to use list ordering and it simply does not reproduce with llc.  I believe Andy knows more about the underlying issue.

I just assumed clang was supported since it’s there is lit.cfg.  Can we perhaps have a new REQUIRES for clang?

Adam

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140428/c3436fb2/attachment.html>


More information about the llvm-commits mailing list