[PATCH] D83928: [ARM][TEST] Add a new test case of add-imm & sub-imm

Ben Shi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 27 07:20:00 PDT 2020


benshi001 added a comment.

In D83928#2175664 <https://reviews.llvm.org/D83928#2175664>, @SjoerdMeijer wrote:

> Although I don't mind at all new tests being added, in this case I am really wondering if the code paths are not exercised by existing tests. And also reading the description:
>
> > This case will show furture optimization of add-imm & sub-imm.
>
> I am wondering what these future optimizations are and if these tests should simply go in with that then?


I expect this test will show improvements in 
https://reviews.llvm.org/D83745
https://reviews.llvm.org/D84100

I hope this test case first be merged, then the above two patches will affect this test.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83928/new/

https://reviews.llvm.org/D83928





More information about the llvm-commits mailing list