[clang] [llvm] Add RunTimeLang to Class and Enumeration DIBuilder functions (PR #72011)

Felipe de Azevedo Piovezan via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 14 16:30:14 PST 2023


================
@@ -424,6 +424,7 @@ namespace llvm {
     /// \param OffsetInBits Member offset.
     /// \param Flags        Flags to encode member attribute, e.g. private
     /// \param Elements     class members.
+    /// \param RunTimeLang  Optional parameter, Objective-C runtime version.
----------------
felipepiovezan wrote:

I think your entire patch would be a lot simpler if this were the last parameter of the function, as you wouldn't need to change as many callsites and have inline comments explaining the parameter. Did you have a motivation in mind for placing the argument here?

https://github.com/llvm/llvm-project/pull/72011


More information about the cfe-commits mailing list