[PATCH] D19884: Fix a bug when hoist spill to a BB with landingpad successor

Wei Mi via llvm-commits llvm-commits at lists.llvm.org
Tue May 3 11:34:58 PDT 2016


wmi created this revision.
wmi added a reviewer: qcolombet.
wmi added subscribers: llvm-commits, davidxl, echristo.
wmi set the repository for this revision to rL LLVM.
Herald added a subscriber: qcolombet.

This is to fix the bug in https://llvm.org/bugs/show_bug.cgi?id=27612.

When spill is hoisted to a BB with landingpad successor, and if the VNI of the spill reg lives into the landingpad successor, the
spill should be inserted before the call which may throw exception. I uses SplitAnalysis::computeLastSplitPoint to get the insert point.





Repository:
  rL LLVM

http://reviews.llvm.org/D19884

Files:
  lib/CodeGen/InlineSpiller.cpp
  lib/CodeGen/SplitKit.cpp
  lib/CodeGen/SplitKit.h
  test/CodeGen/X86/hoist-spill-lpad.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19884.56037.patch
Type: text/x-patch
Size: 9198 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160503/735e100c/attachment.bin>


More information about the llvm-commits mailing list