[PATCH] D57812: [ARM] Add OptMinSize Subtarget feature

Diana Picus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 8 02:34:07 PST 2019


rovka added inline comments.


================
Comment at: lib/Target/ARM/ARMInstrInfo.td:726
   // Ideally this would be an IntImmLeaf, but then we wouldn't have access to
   // the MachineFunction.
   let GISelPredicateCode = [{
----------------
efriedma wrote:
> It should be possible to address this comment from https://reviews.llvm.org/rL351056, since this doesn't require a MachineFunction anymore, but I guess that can be a followup.
My thoughts exactly, I'll look into it next week if nobody else gets to it first :) Thanks!


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57812





More information about the llvm-commits mailing list