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

[Stefan Pintilie via Phabricator via llvm-commits]

stefanp added a comment.

@hfinkel @nemanjai 
Hi, 
Sorry for the late reply. It took me a little while to figure out that I had specified the wrong options for my test when trying to reproduce this....
Anyway, I've verified that what Nemanja said is indeed correct:
no-PIC

  bl      100006b0 <00000017.plt_call.omp_control_tool@@VERSION>                                       
  ld      r2,24(r1)                                                                                    
  nop                                                                                                  
  stw     r3,108(r31)
  nop  

PIC

  bl      100006b0 <00000017.plt_call.omp_control_tool@@VERSION>
  ld      r2,24(r1)
  nop
  nop

The store in the no-PIC version is just saving the return value from `r3`.


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

https://reviews.llvm.org/D56286