[PATCH] D67957: [ARM] Cortex-M4 schedule additions

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 25 01:35:16 PDT 2019


SjoerdMeijer added a comment.

Looks all like very mechanical changes to me. The only thing I am not a big fan of is this `UnsupportedFeatures`, but I understood that there is not really a much better alternative.

The only question I have is related to the `Predicates=[IsWindows]` addition, which is a functional change. Do we need a test for that?


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

https://reviews.llvm.org/D67957





More information about the llvm-commits mailing list