sfertile added inline comments.
================ Comment at: clang/lib/CodeGen/TargetInfo.cpp:4175 namespace { /// PPC32_SVR4_ABIInfo - The 32-bit PowerPC ELF (SVR4) ABI information. class PPC32_SVR4_ABIInfo : public DefaultABIInfo { ---------------- ZarkoCA wrote: > sfertile wrote: > > This name and comment is misleading, the class is used for both SVR4 and > > Darwin, and after this patch AIX. We need to fix the name comment to > > reflect that. > Does this wording of the comment work? Missed updating the class name. Suggested: `PPC32_SVR4_ABIInfo` --> `PowerPC32ABIInfo`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76360/new/ https://reviews.llvm.org/D76360 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits