[llvm] [llvm/llvm-project][Coroutines] Move spill related methods to a Spill utils (PR #107884)

Tyler Nowicki via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 10 12:43:04 PDT 2024


TylerNowicki wrote:

> I didn't check if they are strictly the same line by line. But I believe it is trivial code movement. So LGTM with removing the prefix `[llvm/llvm-project]`.
> 
> But it looks like there were some regression reports for the first issue. Please take a look before landing.

Looks like the test failure is due to a permissions issue.

"C:\a\lld-x86_64-win\llvm-project\llvm\unittests\Support\ProgramTest.cpp(163): error: fs::remove(Twine(LongPath)): did not return errc::success.
error number: 13
error message: permission denied"

https://github.com/llvm/llvm-project/pull/107884


More information about the llvm-commits mailing list