[PATCH] D28409: Use CodegenOpts::less when creating a TargetMachine for clang `-O1`

2017-01-06 Thread Mehdi AMINI via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL291276: Use CodegenOpts::less when creating a TargetMachine for clang `-O1` (authored by mehdi_amini). Changed prior to commit: https://reviews.llvm.org/D28409?vs=83404&id=83408#toc Repository: rL LL

[PATCH] D28409: Use CodegenOpts::less when creating a TargetMachine for clang `-O1`

2017-01-06 Thread Eric Christopher via Phabricator via cfe-commits
echristo accepted this revision. echristo added a comment. This revision is now accepted and ready to land. Sounds fine to me. I want to modify https://reviews.llvm.org/owners/package/1/ at some point, but there's nothing wrong with consistency for now. https://reviews.llvm.org/D28409 __

[PATCH] D28409: Use CodegenOpts::less when creating a TargetMachine for clang `-O1`

2017-01-06 Thread Mehdi AMINI via Phabricator via cfe-commits
mehdi_amini updated this revision to Diff 83404. mehdi_amini added a comment. clang-format https://reviews.llvm.org/D28409 Files: clang/lib/CodeGen/BackendUtil.cpp Index: clang/lib/CodeGen/BackendUtil.cpp === --- clang/lib/Code

[PATCH] D28409: Use CodegenOpts::less when creating a TargetMachine for clang `-O1`

2017-01-06 Thread Mehdi AMINI via Phabricator via cfe-commits
mehdi_amini created this revision. mehdi_amini added reviewers: echristo, chandlerc. mehdi_amini added a subscriber: cfe-commits. Clang was initializing the TargetMachine with CodeGenOpt::Default for https://reviews.llvm.org/owners/package/1/. This change is aligning it on llc: -O0: OptLevel = C