FailedChanges

Summary

  1. Avoid warning of unused variable in release builds. NFC
  2. Revert r301897, "ELF: Set symbol binding to STB_GLOBAL when undefining symbols during LTO." It doesn't matter what binding we store in a non-UsedInRegularObj undefined symbol because we should reset it when we see a real undefined symbol in a combined LTO object. The fact that we weren't doing so before is a bug (PR32899) which is now fixed.
  3. Tests: strengthen CHECK line to avoid picking up stray path. A bot had "-LTO" in its working directory, which matched the regex used in this test. Since the arg is quoted, we can exploit that instead. Still broken if there's a path with a quote in, but I think that's pretty niche.
Revision 302068 by annat:
Avoid warning of unused variable in release builds. NFC
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/Utils/LoopUnrollRuntime.cpp (diff)llvm-revision.src/llvm/trunk/lib/Transforms/Utils/LoopUnrollRuntime.cpp
Revision 302067 by pcc:
Revert r301897, "ELF: Set symbol binding to STB_GLOBAL when undefining symbols during LTO."

It doesn't matter what binding we store in a non-UsedInRegularObj undefined
symbol because we should reset it when we see a real undefined symbol in
a combined LTO object. The fact that we weren't doing so before is a bug
(PR32899) which is now fixed.
Change TypePath in RepositoryPath in Workspace
The file was modified/lld/trunk/ELF/LTO.cpp (diff)llvm-revision.src/lld/trunk/ELF/LTO.cpp
Revision 302066 by Tim Northover:
Tests: strengthen CHECK line to avoid picking up stray path.

A bot had "-LTO" in its working directory, which matched the regex used in this
test. Since the arg is quoted, we can exploit that instead. Still broken if
there's a path with a quote in, but I think that's pretty niche.
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/test/Driver/arch-specific-libdir-rpath.c (diff)llvm-revision.src/cfe/trunk/test/Driver/arch-specific-libdir-rpath.c