[PATCH] D94456: Fix unused variable in CoroFrame.cpp when building Release with GCC 10

Xun Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 25 08:23:41 PST 2021


lxfind added inline comments.


================
Comment at: llvm/lib/Transforms/Coroutines/CoroFrame.cpp:1538
               dyn_cast<CatchSwitchInst>(Pred->getTerminator())) {
+        (void)CS;
         // CleanupPad with a CatchSwitch predecessor: therefore this is an
----------------
AlexeySachkov wrote:
> I wonder, why do we need the second `(void)CS` if we already had one on line 1542 right within the same `if`?
hah looks like this was fixed in https://github.com/llvm/llvm-project/commit/fb98a1be43645c87fff089c4cc9555ca2400268c a day before this commit


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94456/new/

https://reviews.llvm.org/D94456



More information about the llvm-commits mailing list