[PATCH] D56286: [OPENMP] Deal with additional store inserted by Clang under -fno-PIC for PowerPC.

Jonas Hahnfeld via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 1 03:31:51 PST 2019


Hahnfeld added a comment.

In D56286#1411846 <https://reviews.llvm.org/D56286#1411846>, @nemanjai wrote:

> In D56286#1409384 <https://reviews.llvm.org/D56286#1409384>, @hfinkel wrote:
>
> > Is this STW instruction something that instruction scheduling might move? (@nemanjai ?)
>
>
> I am fairly certain that the STW is the store of the value returned by the function so the scheduler should leave it alone. Perhaps @stefanp can verify this.


Well yes, but the return value is never used. So why does the no-PIC version need to store it while the PIC version omits that instruction?


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

https://reviews.llvm.org/D56286





More information about the llvm-commits mailing list