[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