[PATCH] D28150: Move the section name from GlobalObject to the LLVMContext

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added inline comments.


================
Comment at: include/llvm/IR/GlobalObject.h:72
+  }
+  StringRef getSection() const {
+    return hasSection() ? getSectionImpl() : StringRef();
----------------
I'd comment here that the access is not efficient and client is advise to check for `hasSection()` before calling it.


https://reviews.llvm.org/D28150