[PATCH] D108968: [DIArgList] Re-unique after changing operands to fix non-determinism

Alexander Kornienko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 31 14:25:04 PDT 2021


alexfh added inline comments.


================
Comment at: llvm/lib/IR/DebugInfoMetadata.cpp:1646
+  bool Uniq = isUniqued();
+  if (Uniq)
+    // We need to update the uniqueness once the Args are updated since they
----------------
nit: Maybe wrap the body in braces, since it's multiline due to the comment?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108968



More information about the llvm-commits mailing list