XCore target: fix bug in XCoreLowerThreadLocal.cpp

Robert Lytton robert at xmos.com
Wed Oct 2 09:13:17 PDT 2013


I've amended the patch.
For phi node handling the predecessor block is checked for multiple successors and if so the edge is first split.

    XCore target: fix bug in XCoreLowerThreadLocal.cpp
    
    When a ConstantExpr which uses a thread local is part of a PHI node
    instruction, the insruction that replaces the ConstantExpr must
    be inserted in the predecessor block, in front of the terminator instruction.
    If the predecessor block has multiple successors, the edge is first split.

Good to go?

Robert
-------------- next part --------------
A non-text attachment was scrubbed...
Name: PatchThreadLocalPHI2
Type: application/octet-stream
Size: 5399 bytes
Desc: PatchThreadLocalPHI2
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131002/36664331/attachment.obj>


More information about the llvm-commits mailing list