[PATCH] D130786: [clang-repl] Disable execution unittests on unsupported platforms.

2022-07-29 Thread Alan Phipps via Phabricator via cfe-commits
alanphipps added a comment. In D130786#3688503 , @sunho wrote: > @alanphipps I just confirmed that the buildbot failures were fixed by the new > fix https://reviews.llvm.org/rG65c9265f4158. Could you check out if this > fixes the failure on your end

[PATCH] D130786: [clang-repl] Disable execution unittests on unsupported platforms.

2022-07-29 Thread Sunho Kim via Phabricator via cfe-commits
sunho added a comment. @alanphipps I just confirmed that the buildbot failures were fixed by the new fix https://reviews.llvm.org/rG65c9265f4158. Could you check out if this fixes the failure on your end too? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D130786: [clang-repl] Disable execution unittests on unsupported platforms.

2022-07-29 Thread Sunho Kim via Phabricator via cfe-commits
sunho added a comment. In D130786#3688387 , @alanphipps wrote: > I'm still seeing a failure on my downstream Arm compiler on Linux in the unit > tests -- I thought I saw the same failure on the buildbots: > > FAIL: llvm_regressions :: >

[PATCH] D130786: [clang-repl] Disable execution unittests on unsupported platforms.

2022-07-29 Thread Alan Phipps via Phabricator via cfe-commits
alanphipps added a comment. I'm still seeing a failure on my downstream Arm compiler on Linux in the unit tests -- I thought I saw the same failure on the buildbots: FAIL: llvm_regressions :: Clang-Unit/Interpreter/ClangReplInterpreterTests/1/8 3

[PATCH] D130786: [clang-repl] Disable execution unittests on unsupported platforms.

2022-07-29 Thread Sunho Kim via Phabricator via cfe-commits
sunho added a comment. In D130786#3688019 , @probinson wrote: > If you're going to post a patch for review, you really should wait for > someone to review it before you land it. I rushed as bots were breaking on the upstream. Sorry about that.

[PATCH] D130786: [clang-repl] Disable execution unittests on unsupported platforms.

2022-07-29 Thread Paul Robinson via Phabricator via cfe-commits
probinson added a comment. If you're going to post a patch for review, you really should wait for someone to review it before you land it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130786/new/ https://reviews.llvm.org/D130786

[PATCH] D130786: [clang-repl] Disable execution unittests on unsupported platforms.

2022-07-29 Thread Sunho Kim via Phabricator via cfe-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG4191d661c746: [clang-repl] Disable execution unittests on