aaron.ballman added inline comments.
================ Comment at: lib/Basic/Targets/PPC.h:349 LongLongAlign = 32; + UseSignedCharForObjCBool = true; resetDataLayout("E-m:o-p:32:32-f64:32:64-n32"); ---------------- Do you need to specify this? Isn't it handled by the `TargetInfo` constructor? ================ Comment at: lib/Basic/Targets/PPC.h:364 HasAlignMac68kSupport = true; + UseSignedCharForObjCBool = true; resetDataLayout("E-m:o-i64:64-n32:64"); ---------------- Likewise. https://reviews.llvm.org/D29768 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits