[PATCH] D154722: [AArch64] Refactor AArch64InstrInfo::isAsCheapAsAMove (NFC)

Momchil Velikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 24 07:32:13 PDT 2023


chill marked 2 inline comments as done.
chill added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.cpp:823
 
+// FIXME: this implementation should be micro-architecture dependent, so a
+// micro-architecture target hook should be introduced here in future.
----------------
dmgreen wrote:
> What do you mean by micro-architecture dependent?
Existing comment, perhaps means use a sub-target hook, instead of a target hook.


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

https://reviews.llvm.org/D154722



More information about the llvm-commits mailing list