[PATCH] D125846: [StackProtector] Allow targets to specify an instruction is part of terminator sequence

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 18 15:12:19 PDT 2022


nemanjai added a comment.

In D125846#3521120 <https://reviews.llvm.org/D125846#3521120>, @aemerson wrote:

> Can the test be reduced? Do you really need a `main()`?

Good point. In the original test case I reduced `func1()` is `static` so it needed the caller. I should be able to change the linkage of the function and remove `main()`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125846



More information about the llvm-commits mailing list