SuccessChanges

Summary

  1. [CodeGen] Add "noreturn" attirbute to _Unwind_Resume (details)
  2. Revert "[InstCombine] Check inbounds in load/store of gep null transform (PR48577)" (details)
Commit e0751234ef0df733032b777ed0d993a490121855 by ybrevnov
[CodeGen] Add "noreturn" attirbute to _Unwind_Resume

Currently 'resume' is lowered to _Unwind_Resume with out "noreturn" attribute. Semantically _Unwind_Resume  library call is expected to never return and should be marked as such. Though I didn't find any changes in behavior of existing tests there will be a difference once https://reviews.llvm.org/D79485 lands.

I was not able to come up with the test case anything better than just checking for presence of "noreturn" attribute. Please let me know if there is a better way to test the change.

Reviewed By: xbolva00

Differential Revision: https://reviews.llvm.org/D93682
The file was addedllvm/test/CodeGen/Generic/dwarf_eh_resume.ll
The file was modifiedllvm/lib/CodeGen/DwarfEHPrepare.cpp
The file was modifiedllvm/test/CodeGen/AArch64/GlobalISel/irtranslator-invoke-probabilities.ll
Commit ef2f843347baca1ece69066a3c3a7afa45532079 by nikita.ppv
Revert "[InstCombine] Check inbounds in load/store of gep null transform (PR48577)"

This reverts commit 899faa50f206073cdd8eeaaa130ffa15f850e656.

Upon further consideration, this does not fix the right issue.
Doing this fold for non-inbounds GEPs is legal, because the
resulting pointer is still based-on null, which has no associated
address range, and as such and access to it is UB.

https://bugs.llvm.org/show_bug.cgi?id=48577#c3
The file was modifiedllvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp
The file was modifiedllvm/test/Transforms/InstCombine/load.ll
The file was modifiedllvm/test/Transforms/InstCombine/store.ll