[PATCH] D107196: [AArch64InstPrinter] Change printAddSubImm to only add imm value comment when shifted

Tim Northover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 3 01:21:50 PDT 2021


t.p.northover added a comment.

In D107196#2917773 <https://reviews.llvm.org/D107196#2917773>, @MaskRay wrote:

> Consider committing the test update separately from the code patch, so that the functional change has very few lines of update.

Wouldn't that mean a revision with lots of tests failing? I think a single large commit would be better. People can grep out changes in llvm/test if they're not interested in that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107196



More information about the llvm-commits mailing list