[PATCH] D106308: [Local] Do not introduce a new `llvm.trap` before `unreachable`

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 19 13:40:39 PDT 2021


jdoerfert created this revision.
jdoerfert added reviewers: lebedev.ri, efriedma.
Herald added subscribers: ormris, lxfind, okura, kuter, zzheng, bollu, hiraditya.
jdoerfert requested review of this revision.
Herald added a reviewer: sstefan1.
Herald added a reviewer: baziotis.
Herald added a subscriber: bbn.
Herald added a project: LLVM.

This is the second attempt to remove the `llvm.trap` insertion after

  https://reviews.llvm.org/rGe14e7bc4b889dfaffb7180d176a03311df2d4ae6

reverted the first one. It is not clear what the exact issue was back
then and it might already be gone by now, it has been >5 years after
all.

Replaces D106299 <https://reviews.llvm.org/D106299>.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D106308

Files:
  llvm/include/llvm/Transforms/Utils/Local.h
  llvm/lib/CodeGen/WinEHPrepare.cpp
  llvm/lib/Transforms/Coroutines/CoroSplit.cpp
  llvm/lib/Transforms/Coroutines/Coroutines.cpp
  llvm/lib/Transforms/IPO/Attributor.cpp
  llvm/lib/Transforms/IPO/PruneEH.cpp
  llvm/lib/Transforms/Scalar/SCCP.cpp
  llvm/lib/Transforms/Utils/InlineFunction.cpp
  llvm/lib/Transforms/Utils/Local.cpp
  llvm/lib/Transforms/Utils/LoopSimplify.cpp
  llvm/lib/Transforms/Utils/LoopUnroll.cpp
  llvm/lib/Transforms/Utils/LoopUtils.cpp
  llvm/unittests/Transforms/Utils/LocalTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D106308.359897.patch
Type: text/x-patch
Size: 10976 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210719/73585ead/attachment.bin>


More information about the llvm-commits mailing list