Details
Details
Diff Detail
Diff Detail
Event Timeline
llvm/lib/Support/FileCheck.cpp | ||
---|---|---|
345 | I fixed a build failure with -Werror where GCC was the host compiler by doing the reverse change. So this change would most likely result on build failure when host compiler is GCC <9.x |
llvm/tools/llvm-objcopy/CopyConfig.cpp | ||
---|---|---|
573 | khm, wouldn't it break the builds using MSVC ? https://gcc.godbolt.org/z/esJH6J |
Comment Actions
Some past reference: http://lists.llvm.org/pipermail/llvm-dev/2018-September/126471.html
Comment Actions
Agree with all these comments -- I'm no expert on when and when not to std::move, but this is undoing changes I've made to fix buildbots on different compilers.
Comment Actions
I agree. This is probably more aggressive than I anticipated. I am looking at them and trying to figure out which and if some of them can stay.
I fixed a build failure with -Werror where GCC was the host compiler by doing the reverse change. So this change would most likely result on build failure when host compiler is GCC <9.x