[PATCH] D96250: [CostModel] An extending load to illegal type is not free.

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 8 06:18:35 PST 2021


sdesmalen added a comment.

In D96250#2548425 <https://reviews.llvm.org/D96250#2548425>, @dmgreen wrote:

> The v8-main/base costs are a little odd, as it's not an architecture that has vectors. But the extends of the loads should probably be free once it's scalarized, and the loads are already more expensive.

Ah that explains the scalar code I saw when I compiled some of the examples with `llc`. The zero/sign extends were indeed folded into the scalar loads.

Thanks for the review!


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

https://reviews.llvm.org/D96250



More information about the llvm-commits mailing list