[PATCH] D81924: [PowerPC] Allow constrained FP intrinsics in mightUseCTR

Qiu Chaofan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 19 00:59:47 PDT 2020


qiucf added a comment.

In D81924#2189996 <https://reviews.llvm.org/D81924#2189996>, @steven.zhang wrote:

> Can you add checker for the bdnz ?

We don't think any loop containing calls is profitable to transform:

  // We don't want to spill/restore the counter register, and so we don't
  // want to use the counter register if the loop contains calls.

And it seems PPC won't set hardware loops info properly if it's not profitable. So when forcing it to use hardware loops, it would crash.


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

https://reviews.llvm.org/D81924



More information about the llvm-commits mailing list