[PATCH] D110258: [AArch64] Always add -tune-cpu argument to -cc1 driver

David Sherwood via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 27 06:29:23 PDT 2021


david-arm added a comment.

Hi @dmgreen, would you be happy for me to do the splitting-out of arch and tuning features in a separate follow-on patch? I think it's a good idea and I don't object to doing it, but I'm not sure that it really needs to hold up this initial patch? I personally think it makes sense to live in a separate patch because it seems riskier in terms of possible effects on performance. As far as I understand it, this isn't a functional requirement, but more for completeness.


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

https://reviews.llvm.org/D110258



More information about the cfe-commits mailing list