[PATCH] D113301: [NFC][llvm-libtool-darwin]Remove unnecessary conditionals around errors.
Daniel RodrÃguez Troitiño via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 5 13:15:22 PDT 2021
drodriguez accepted this revision.
drodriguez added inline comments.
This revision is now accepted and ready to land.
================
Comment at: llvm/tools/llvm-libtool-darwin/llvm-libtool-darwin.cpp:508
llvm::stable_sort(*Slices);
- if (Error E = writeUniversalBinary(*Slices, OutputFile))
- return E;
+ return writeUniversalBinary(*Slices, OutputFile);
}
----------------
For this last case, if you want to remove the `else` branch, I think it will fit more with the code style: https://llvm.org/docs/CodingStandards.html#don-t-use-else-after-a-return
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113301/new/
https://reviews.llvm.org/D113301
More information about the llvm-commits
mailing list