[PATCH] D20271: [TM] Start collapsing LLVMTargetMachine into TargetMachine

[Davide Italiano via llvm-commits]

davide added inline comments.

================
Comment at: lib/Target/TargetMachine.cpp:151
@@ -150,3 +151,3 @@
 
 TargetIRAnalysis TargetMachine::getTargetIRAnalysis() {
   return TargetIRAnalysis([this](const Function &F) {
----------------
rafael wrote:
> Can you change this to be non-virtual now?
Not entirely sure because PPC and Hexagon still override that? Can we somehow avoid it?
```
../lib/Target/PowerPC/PPCTargetMachine.h:48:42: error: only virtual member functions can be marked 'override'
  TargetIRAnalysis getTargetIRAnalysis() override;
                                         ^~~~~~~~
```


http://reviews.llvm.org/D20271