aboutsummaryrefslogtreecommitdiff
path: root/flang/lib/Frontend/CompilerInvocation.cpp
diff options
context:
space:
mode:
authorMircea Trofin <mtrofin@google.com>2024-05-15 10:34:47 -0700
committerGitHub <noreply@github.com>2024-05-15 10:34:47 -0700
commit217668f641e82f901645f428ae0d07a3c01e9a8a (patch)
treed2c428d07a420c27b3b7aea38d6b9be55ac9ea9e /flang/lib/Frontend/CompilerInvocation.cpp
parentd92c67784f21063d6334a009dbf4f9e0f8217b41 (diff)
downloadllvm-217668f641e82f901645f428ae0d07a3c01e9a8a.zip
llvm-217668f641e82f901645f428ae0d07a3c01e9a8a.tar.gz
llvm-217668f641e82f901645f428ae0d07a3c01e9a8a.tar.bz2
[nfc] Allow forwarding `Error` returns from `Expected` callers (#92208)
On a few compilers (clang 11/12 for example [1]), the following does not result in a copy elision, and since `Error`'s copy dtor is elided, results in a compile error: ``` Expect<Something> foobar() { ... if (Error E = aCallReturningError()) return E; ... } ``` Moving `E` would, conversely, result in the pessimizing-move warning on more recent clangs ("moving a local object in a return statement prevents copy elision") We just need to make the `Expected` ctor taking an `Error` take it as a r-value reference. [1] https://lab.llvm.org/buildbot/#/builders/54/builds/10505
Diffstat (limited to 'flang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions