[PATCH] D50141: Add errors for tiny codemodel on targets other than AArch64

Tue Nov 13 10:04:33 PST 2018

dmgreen updated this revision to Diff 173869.
dmgreen marked an inline comment as done.
dmgreen added a comment.

I'm happy to change stuff around. I believe it's the way it is at the moment because of the place this is called, half way into a constructor (making virtuals difficult) and some archs requiring different pieces of info (like aarch64 being different for JIT's). I figured if in doubt, don't change too much at once.

This is a rebase, plus your suggested comment. Let me know if this is OK as-is, or I should change it up more.

https://reviews.llvm.org/D50141

Files:
  include/llvm/Target/TargetMachine.h
  lib/Target/AArch64/AArch64TargetMachine.cpp
  lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
  lib/Target/ARC/ARCTargetMachine.cpp
  lib/Target/ARM/ARMTargetMachine.cpp
  lib/Target/AVR/AVRTargetMachine.cpp
  lib/Target/BPF/BPFTargetMachine.cpp
  lib/Target/Hexagon/HexagonTargetMachine.cpp
  lib/Target/Lanai/LanaiTargetMachine.cpp
  lib/Target/MSP430/MSP430TargetMachine.cpp
  lib/Target/Mips/MipsTargetMachine.cpp
  lib/Target/NVPTX/NVPTXTargetMachine.cpp
  lib/Target/Nios2/Nios2TargetMachine.cpp
  lib/Target/PowerPC/PPCTargetMachine.cpp
  lib/Target/RISCV/RISCVTargetMachine.cpp
  lib/Target/Sparc/SparcTargetMachine.cpp
  lib/Target/SystemZ/SystemZTargetMachine.cpp
  lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp
  lib/Target/X86/X86TargetMachine.cpp
  lib/Target/XCore/XCoreTargetMachine.cpp
  test/CodeGen/ARM/codemodel.ll
  test/CodeGen/Lanai/codemodel.ll
  test/CodeGen/PowerPC/codemodel.ll
  test/CodeGen/SPARC/codemodel.ll
  test/CodeGen/SystemZ/codemodel.ll
  test/CodeGen/X86/codemodel.ll
  test/CodeGen/XCore/codemodel.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50141.173869.patch
Type: text/x-patch
Size: 28755 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181113/9ca7ccbe/attachment-0001.bin>