[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