mgrang added a comment. In https://reviews.llvm.org/D35529#812427, @mstorsjo wrote:
> Wouldn't it make more sense to set this by default within LLVM, where it's > already enabled by default for darwin? > > Currently there's this code there: > > AArch64Subtarget::AArch64Subtarget(const Triple &TT, const std::string &CPU, > const std::string &FS, > const TargetMachine &TM, bool > LittleEndian) > : AArch64GenSubtargetInfo(TT, CPU, FS), ReserveX18(TT.isOSDarwin()), > Thanks for pointing this out. This certainly seems a cleaner way/place to do this. Lemme give it a shot. https://reviews.llvm.org/D35529 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
