rsmith added inline comments.

================
Comment at: test/SemaCXX/source_location.cpp:10-35
+struct source_location {
+private:
+  unsigned int __m_line = 0;
+  unsigned int __m_col = 0;
+  const char *__m_file = nullptr;
+  const char *__m_func = nullptr;
+public:
----------------
This seems suboptimal. It would seem better for the compiler to generate a 
global containing the relevant data and to represent a `source_location` as a 
pointer to such a value. We should also try to minimize the number of 
relocations necessary to build a `source_location` object, since such 
constructions are likely to be extremely common in some codebases. We should 
also keep in mind that we're likely to want to add fields to `source_location` 
in future, so designing it in a way that avoids an ABI break for such cases 
would be preferable.

How long has GCC supported this?


https://reviews.llvm.org/D37035



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to