[PATCH] D34347: [PowerPC] fix potential verification error on __tls_get_addr

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 21 07:38:13 PDT 2017


hfinkel added inline comments.


================
Comment at: lib/Target/PowerPC/PPCTLSDynamicCall.cpp:112
         // registers are already saved when the SDNode (e.g. PPCaddiTlsgdLAddr)
         // gets translated to the pseudo instruction (e.g. ADDItlsgdLADDR).
+        if (NeedFence)
----------------
This comment does not explain the "Here, ADJCALLSTACKUP and ADJCALLSTACKDOWN are created as a fence for instruction scheduling to avoid _tls_get_addr is scheduled before mflr in the prologue". I realize that this is a pre-existing issue, but while you're here, we should document this better.



https://reviews.llvm.org/D34347





More information about the llvm-commits mailing list