================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to