[PATCH] D124203: [clang][safestack] Remove triple from stack usage test

2022-04-21 Thread Paul Kirth via Phabricator via cfe-commits
paulkirth added a comment. unfortunately it breaks many other tests, so we probably need to revert this Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124203/new/ https://reviews.llvm.org/D124203 ___ cfe-

[PATCH] D124203: [clang][safestack] Remove triple from stack usage test

2022-04-21 Thread Vlad Vereschaka via Phabricator via cfe-commits
vvereschaka added a comment. Thank you @paulkirth. I have tested these changes locally on the builder. It works fine. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124203/new/ https://reviews.llvm.org/D124203 _

[PATCH] D124203: [clang][safestack] Remove triple from stack usage test

2022-04-21 Thread Paul Kirth via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG414f84ba29d9: [clang][safestack] Remove triple from stack usage test (authored by paulkirth). Repository: rG LLVM Github Monorepo CHANGES SINCE L

[PATCH] D124203: [clang][safestack] Remove triple from stack usage test

2022-04-21 Thread Paul Kirth via Phabricator via cfe-commits
paulkirth created this revision. Herald added a project: All. paulkirth requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Supplying the target triple caused breakeages for compilers that don't support the supplied triple. Repository: rG L