[PATCH] D22648: [PM] Port NaryReassociate to the new PM

Wei Mi via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 21 14:56:15 PDT 2016


>> ================
>> Comment at: test/Transforms/NaryReassociate/nary-mul.ll:2
>> @@ -1,2 +1,3 @@
>>  ; RUN: opt < %s -nary-reassociate -S | FileCheck %s
>> +; RUN: opt < %s -passes='nary-reassociate' -S | FileCheck %s
>>
>> ----------------
>> davidxl wrote:
>> > How about other tests in the same dir?
>> I saw the guildeline in https://reviews.llvm.org/rL272505 said one test
>> changed was enough. I just tried other tests and they passed. Do you want me
>> to include them?
>>
>
> The point is that when old PM is deprecated, the tests need to be changed
> anyway. THere is also a chance more tests can uncover issues.
>
> David
>

Ok, I will add those tests.

Wei.


More information about the llvm-commits mailing list