[PATCH] D105207: [debuginfo][lsr] SCEV-based salvaging for LoopStrengthReduction

Chris Jackson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 27 06:05:07 PDT 2021


chrisjackson updated this revision to Diff 362003.
chrisjackson added a comment.

- Add 'REQUIRES' clause to lit tests to prevent failure on aarch64.
- A failure on buildbot sanitixer-X86_64-linux indicates that SCEVUknown::getValue() can return nullptr. Account for this by returning false and abandoning the salvage attempt.


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

https://reviews.llvm.org/D105207

Files:
  llvm/include/llvm/Transforms/Utils/ScalarEvolutionExpander.h
  llvm/lib/Transforms/Scalar/LoopStrengthReduce.cpp
  llvm/lib/Transforms/Utils/ScalarEvolutionExpander.cpp
  llvm/test/Transforms/LoopStrengthReduce/dbg-preserve-0.ll
  llvm/test/Transforms/LoopStrengthReduce/dbg-preserve-2.ll
  llvm/test/Transforms/LoopStrengthReduce/debuginfo-scev-salvage-0.ll
  llvm/test/Transforms/LoopStrengthReduce/debuginfo-scev-salvage-1.ll
  llvm/test/Transforms/LoopStrengthReduce/debuginfo-scev-salvage-2.ll
  llvm/test/Transforms/LoopStrengthReduce/debuginfo-scev-salvage-3.ll
  llvm/test/Transforms/LoopStrengthReduce/debuginfo-scev-salvage-4.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D105207.362003.patch
Type: text/x-patch
Size: 61108 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210727/869d454a/attachment-0001.bin>


More information about the llvm-commits mailing list