[PATCH] D42618: [LLVM-C] Add Accessors For A Module's Source File Name

whitequark via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 28 06:00:44 PST 2018


whitequark added inline comments.


================
Comment at: lib/IR/Core.cpp:240
+  *Len = Str.length();
+  return Str.c_str();
+}
----------------
Shouldn't the result be returned in a newly allocated buffer here?


Repository:
  rL LLVM

https://reviews.llvm.org/D42618





More information about the llvm-commits mailing list