eugenis accepted this revision.
eugenis added a comment.

LGTM



================
Comment at: llvm/lib/CodeGen/GlobalMerge.cpp:655
 
+    // Don't merged tagged globals, as each global should have its own unique
+    // memory tag at runtime. TODO(hctim): This can be relaxed: constant 
globals
----------------
typo: "merge"


================
Comment at: llvm/lib/Target/AArch64/AArch64GlobalsTagging.cpp:75
+  if (SizeInBytes != NewSize) {
+    DataLayout NewLayout = M.getDataLayout();
+
----------------
unused variable


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133392

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D133392: [MTE] Ad... Mitch Phillips via Phabricator via cfe-commits
    • [PATCH] D133392: [MT... Evgenii Stepanov via Phabricator via cfe-commits

Reply via email to