[PATCH] D71773: [AArch64][SVE] Update the definition of AdvSIMD_GatherLoad_VecTorBase_Intrinsic

Andrzej Warzynski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 2 03:02:24 PST 2020


andwar added a subscriber: eli.friedman.
andwar added a comment.

Thank you for taking a look @eli.friedman. I've kept `int_aarch64_sve_ld1_gather` and `int_aarch64_sve_ld1_gather_imm` separate to better reflect the difference in the semantics of the corresponding instructions. I think that merging the two would obfuscate that separation a bit  (though reduce code, which is usually a good thing). Keeping it as is would be my preference, but I might be missing the bigger picture?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71773





More information about the llvm-commits mailing list