[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