[PATCH] D78147: [LICM] Try to merge debug locations when sinking

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 14 15:45:45 PDT 2020


vsk added inline comments.


================
Comment at: llvm/include/llvm/IR/DebugInfoMetadata.h:1554
+  static
+  const DILocation *getMergedLocations(std::vector<const DILocation *> &Locs);
+
----------------
aprantl wrote:
> Could this be an ArrayRef or MutableArrayRef?
Nice catch, ArrayRef should work, I think.


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

https://reviews.llvm.org/D78147





More information about the llvm-commits mailing list