[PATCH] D131388: [docs] Add "C++20 Modules"

Daniel Ruoso via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 12 08:53:05 PDT 2022


ruoso added inline comments.


================
Comment at: clang/docs/CPlusPlus20Modules.rst:309
+
+Remember to compile and link BMIs
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
----------------
I think this is a bit confusing. The BMI is not linked...

Maybe something like: "Remember that modules still have an object counterpart to the BMI". 

Specifically, it may be important to make a distinction between when the compiler is invoked to produce the bmi and when it's invoked to produce the object, and that you use the bmi when translating code that imports it, and the object when linking.



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

https://reviews.llvm.org/D131388



More information about the cfe-commits mailing list