[PATCH] D52125: X86, AArch64, ARM: Do not attach debug location to spill/reload instructions

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 14 15:03:50 PDT 2018


MatzeB created this revision.
Herald added subscribers: llvm-commits, chrib, kristof.beyls, mcrosier.
Herald added a reviewer: javed.absar.

Spill/reload instructions are artificially generated by the compiler and
have no relation to the original source code. So the best thing to do is
not attach any debug location to them (instead of just taking the next
debug location we find on following instructions).


Repository:
  rL LLVM

https://reviews.llvm.org/D52125

Files:
  lib/Target/AArch64/AArch64InstrInfo.cpp
  lib/Target/ARM/ARMBaseInstrInfo.cpp
  lib/Target/X86/X86InstrInfo.cpp
  test/DebugInfo/X86/fission-ranges.ll
  test/DebugInfo/X86/parameters.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52125.165596.patch
Type: text/x-patch
Size: 10801 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180914/2e986151/attachment.bin>


More information about the llvm-commits mailing list