devnexen added inline comments.
================ Comment at: lib/Driver/XRayArgs.cpp:56 + << (std::string(XRayInstrumentOption) + " only on " + "FreeBSD x86_64"); + } ---------------- krytarowski wrote: > krytarowski wrote: > > I think it's better to not hardcode `x86_64` here in a message, we will > > need change it with new CPUs. > I think we can reuse the existing above code here as well: > > `<< (std::string(XRayInstrumentOption) + " on " + Triple.str());` True https://reviews.llvm.org/D43378 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
