[PATCH] D119296: KCFI sanitizer

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 18 16:47:12 PDT 2022


MaskRay added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64AsmPrinter.cpp:330
+  // comparing the hashes.
+  unsigned ScratchRegs[] = {AArch64::W16, AArch64::W17};
+
----------------
const?


================
Comment at: llvm/lib/Target/AArch64/AArch64AsmPrinter.cpp:331
+  unsigned ScratchRegs[] = {AArch64::W16, AArch64::W17};
+
+  if (AddrReg == AArch64::XZR) {
----------------
delete blank line


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119296



More information about the llvm-commits mailing list