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

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 6 03:09:13 PST 2019


fhahn added a reviewer: fhahn.
fhahn added a comment.

Hi Sam! There are already 2 function attributes indicating whether to optimize for size (minsize, optsize). What is the benefit of adding another target-specific one? I agree that exposing optForMinSize() in STI makes sense, but cannot we just use the existing minsize attribute?


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

https://reviews.llvm.org/D57812





More information about the llvm-commits mailing list