[PATCH] D30046: [globalisel] Change LLT constructor string into an LLT subclass that knows how to generate it.

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 3 04:29:51 PST 2017


dsanders updated this revision to Diff 90458.
dsanders added a comment.

Bring back the original intent of the patch which went missing in diff 89525.

Update with the modulemap solution. The constructor solution is also viable but
when I brought back the missing code I noticed that MVTToLLT() ought to be
implemented with that constructor once we can import all SelectionDAG types
and that means we'll have to bring it back to Support by then.

Tested with module support enabled.


https://reviews.llvm.org/D30046

Files:
  include/llvm/CodeGen/LowLevelType.h
  include/llvm/Support/LowLevelTypeImpl.h
  include/llvm/module.modulemap
  lib/CodeGen/GlobalISel/IRTranslator.cpp
  lib/CodeGen/LowLevelType.cpp
  lib/Support/CMakeLists.txt
  lib/Support/LowLevelType.cpp
  lib/Target/AArch64/AArch64CallLowering.cpp
  lib/Target/AMDGPU/AMDGPUCallLowering.cpp
  lib/Target/X86/X86CallLowering.cpp
  unittests/CodeGen/LowLevelTypeTest.cpp
  utils/TableGen/GlobalISelEmitter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30046.90458.patch
Type: text/x-patch
Size: 24410 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170303/7592726b/attachment.bin>


More information about the llvm-commits mailing list