[PATCH] D35334: ThinLTO Minimized Bitcode File Size Reduction
Mehdi AMINI via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 12 18:15:34 PDT 2017
mehdi_amini added inline comments.
================
Comment at: include/llvm/Bitcode/BitcodeWriter.h:89
+ ///
+ /// Index is required non-null.
+ ///
----------------
haojiewang wrote:
> mehdi_amini wrote:
> > Use a const reference then?
> But M is actually also required non-null. I just used the same args list with writeModule. So here using const references instead of pointer is better?
I believe references carry the non_null information explicitly as part of the API.
https://reviews.llvm.org/D35334
More information about the llvm-commits
mailing list