[llvm-commits] [PATCH] Modify tablegen to use constants when there are more than 32 subtarget features

Zhang, Andy andy.zhang at intel.com
Tue Jan 3 09:23:08 PST 2012


Currently, TableGen uses an anonymous enum for subtarget features. When
compiling with Visual C++, enums are always 'int's, so with more than 32 subtarget
features, some values will overflow.

The attached patch modifies TableGen to emit 'const uint64_t' for subtarget
features when there are more than 32 of them. It was split off from my
earlier patch for Atom optimizations, as requested.

If this looks OK, could it be committed to trunk?

Thanks,
Andy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: tblgen.diff
Type: application/octet-stream
Size: 1695 bytes
Desc: tblgen.diff
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120103/56f04a98/attachment.obj>


More information about the llvm-commits mailing list