[PATCH] D70966: [ARM] Teach the Arm cost model that a Shift can be folded into other instructions

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 8 07:08:58 PST 2019


spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfo.h:904
   /// cases or optimizations based on those values.
+  /// \p CtxI is the optional original context instruction, if one exists, to
+  /// provide even more information.
----------------
typo: CtxI -> CxtI
That's why I don't like that particular abbreviation - it doesn't "sound" right. :)


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

https://reviews.llvm.org/D70966





More information about the llvm-commits mailing list