[PATCH] D132349: [AArch64][NFC] Merge some common ISD node when setOperationAction

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added a comment.

Hi @Jimerlife is there a specific reason why you're refactoring this? This patch may reduce the number of lines of code, but it is arguable whether the new version is more readable or better. Code-refactoring creates merge-problems for people needing to maintain downstream branches, so unless there is a clear benefit to the code it is preferable to keep this to a minimum.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132349