[PATCH] D86528: [4/5] [MC] [Win64EH] Fill in FuncletOrFuncEnd if missing

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 26 17:26:55 PDT 2020


efriedma added inline comments.


================
Comment at: llvm/lib/MC/MCWin64EH.cpp:702
+  if (!info->FuncletOrFuncEnd &&
+      Streamer.getCurrentSectionOnly() == info->TextSection)
+    info->FuncletOrFuncEnd = Streamer.emitCFILabel();
----------------
Instead of checking `Streamer.getCurrentSectionOnly() == info->TextSection`, can we just `Streamer.SwitchSection(info->TextSection)`?


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

https://reviews.llvm.org/D86528



More information about the llvm-commits mailing list