[PATCH] D71954: [PowerPC] Change default for unaligned FP access for older subtargets

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 28 08:53:30 PST 2019


hfinkel accepted this revision as: hfinkel.
hfinkel added a comment.
This revision is now accepted and ready to land.

In D71954#1797765 <https://reviews.llvm.org/D71954#1797765>, @nemanjai wrote:

> Add the feature to the BG cores as well.


BG did allow this, but IIRC, only some finite number of times (before it really slowed down by logging a lot from the kernel). Maybe this was just for the vector types, but regardless, I'd leave this off for the BG cores.

Otherwise, LGTM.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71954





More information about the llvm-commits mailing list