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

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 27 06:11:15 PDT 2020


SjoerdMeijer added a comment.

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?


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

https://reviews.llvm.org/D83928





More information about the llvm-commits mailing list