[PATCH] D159357: [AArch64] Move PAuth codegen down the machine pipeline

Tomas Matheson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 5 05:35:51 PDT 2023


tmatheson added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64PointerAuth.cpp:157
+      auto It = MI.getIterator();
+      bool HasWinCFI = false;
+      switch (MI.getOpcode()) {
----------------
Looks like HasWinCFI is not used for anything but the asserts below, it could be removed. It has always been a confusing part of this code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159357



More information about the llvm-commits mailing list