<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 9, 2016 at 12:29 PM, Matt Arsenault via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">arsenm created this revision.<br>
arsenm added a reviewer: tstellarAMD.<br>
arsenm added a subscriber: llvm-commits.<br>
Herald added subscribers: arsenm, MatzeB.<br>
<br>
Move a few functions only used by R600 to R600 specific code,<br>
fix header macros to stop using R600, mark classes as final.<br></blockquote><div><br></div><div>What's the reason for marking the classes final? We don't usually go out of our way to do that, except for -Wnon-virtual-dtor silencing (in my experience/practice) - so perhaps it's that case? (just not clear to me from the commit/code (without going & looking at the hierarchies, etc)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<a href="http://reviews.llvm.org/D18010" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18010</a><br>
<br>
Files:<br>
  lib/Target/AMDGPU/AMDGPU.h<br>
  lib/Target/AMDGPU/AMDGPUAsmPrinter.h<br>
  lib/Target/AMDGPU/AMDGPUISelLowering.cpp<br>
  lib/Target/AMDGPU/AMDGPUISelLowering.h<br>
  lib/Target/AMDGPU/AMDGPUInstrInfo.h<br>
  lib/Target/AMDGPU/AMDGPUIntrinsicInfo.h<br>
  lib/Target/AMDGPU/AMDGPUMCInstLower.h<br>
  lib/Target/AMDGPU/AMDGPUMachineFunction.h<br>
  lib/Target/AMDGPU/AMDGPURegisterInfo.h<br>
  lib/Target/AMDGPU/AMDGPUTargetMachine.cpp<br>
  lib/Target/AMDGPU/AMDGPUTargetMachine.h<br>
  lib/Target/AMDGPU/AMDGPUTargetTransformInfo.h<br>
  lib/Target/AMDGPU/InstPrinter/AMDGPUInstPrinter.h<br>
  lib/Target/AMDGPU/MCTargetDesc/AMDGPUELFStreamer.h<br>
  lib/Target/AMDGPU/MCTargetDesc/AMDGPUFixupKinds.h<br>
  lib/Target/AMDGPU/MCTargetDesc/AMDGPUMCAsmInfo.h<br>
  lib/Target/AMDGPU/MCTargetDesc/AMDGPUMCCodeEmitter.h<br>
  lib/Target/AMDGPU/MCTargetDesc/AMDGPUMCTargetDesc.h<br>
  lib/Target/AMDGPU/MCTargetDesc/AMDGPUTargetStreamer.h<br>
  lib/Target/AMDGPU/R600Defines.h<br>
  lib/Target/AMDGPU/R600ISelLowering.cpp<br>
  lib/Target/AMDGPU/R600ISelLowering.h<br>
  lib/Target/AMDGPU/R600InstrInfo.h<br>
  lib/Target/AMDGPU/R600MachineFunctionInfo.h<br>
  lib/Target/AMDGPU/R600MachineScheduler.h<br>
  lib/Target/AMDGPU/R600RegisterInfo.h<br>
  lib/Target/AMDGPU/SIDefines.h<br>
  lib/Target/AMDGPU/SIISelLowering.h<br>
  lib/Target/AMDGPU/SIInstrInfo.h<br>
  lib/Target/AMDGPU/SIMachineFunctionInfo.h<br>
  lib/Target/AMDGPU/SIMachineScheduler.h<br>
  lib/Target/AMDGPU/SIRegisterInfo.h<br>
<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div></div>