[PATCH] D85670: [Instruction] Add updateLocationAfterHoist helper

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 23 16:48:20 PDT 2020


vsk updated this revision to Diff 293896.
vsk added a comment.

- To drop the location of a call within a function with no subprogram, set an empty DebugLoc instead of trying to preserve scope/inlinedAt info. Update a unit test accordingly.


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

https://reviews.llvm.org/D85670

Files:
  llvm/include/llvm/IR/Instruction.h
  llvm/lib/IR/DebugInfo.cpp
  llvm/lib/Transforms/Scalar/GVN.cpp
  llvm/lib/Transforms/Scalar/LICM.cpp
  llvm/test/DebugInfo/Generic/licm-hoist-debug-loc.ll
  llvm/test/Transforms/GVN/PRE/phi-translate.ll
  llvm/test/Transforms/LICM/hoisting-preheader-debugloc.ll
  llvm/unittests/IR/InstructionsTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D85670.293896.patch
Type: text/x-patch
Size: 10192 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200923/86653ac5/attachment.bin>


More information about the llvm-commits mailing list