aaron.ballman added a comment.

In D126984#3581829 <https://reviews.llvm.org/D126984#3581829>, @xbolva00 wrote:

> I was thinking about this again and I am more and more unsure about this 
> feature. -Os/-Oz is something more than just some attribute.

That's not stopped us from exposing attributes like `minsize` (and you proposed 
`optsize` as well) which get described in the same terms as -Os/-Oz in our 
documentation, so this ship has somewhat sailed.

(FWIW, I don't have a strong opinion on the attributes we expose here. I do 
have strong opinions about increasing the maintenance burdens by adding more 
related attributes without carefully considering their semantic and code gen 
interactions.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126984

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to