[PATCH] D44687: [SchedModel] Remove instregex entries that don't match any instructions (WIP)

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 20 12:10:10 PDT 2018


RKSimon added a comment.

In https://reviews.llvm.org/D44687#1043542, @craig.topper wrote:

> I'd like to see the loops merged to get rid of the nested std::pairs. But yeah I don't want to get in the way of fixing all the targets.
>
> The X86 changes look to me. That FMA was one in Zen is interesting. Does the the ZnWriteResFpuPair for FMA have the right information that prevented that from affecting tests?


Yes, the override is probably redundant - see PR35548


Repository:
  rL LLVM

https://reviews.llvm.org/D44687





More information about the llvm-commits mailing list