yaron.keren added a comment. LGTM after fixing the comment.
================ Comment at: tools/clang/lib/Basic/Targets.cpp:4005 @@ +4004,3 @@ + MacroBuilder &Builder) const override { + X86_32TargetInfo::getTargetDefines(Opts, Builder); + Builder.defineMacro("__x86_64__"); ---------------- This should be X86_64TargetInfo::getTargetDefines(Opts, Builder); Repository: rL LLVM http://reviews.llvm.org/D11349 _______________________________________________ cfe-commits mailing list cfe-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits