[PATCH] D71210: [MC] Widen the functional unit type from 32 to 64 bits.

James Molloy via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 10 03:09:02 PST 2019


jmolloy accepted this revision.
jmolloy added a comment.
This revision is now accepted and ready to land.

In general anything moving from 32-bit to 64-bit bitmasks in the backend LGTM. I'd even go further and suggest using std::bitset to remove this problem Once And For All, but that would require significantly more benchmarking.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71210/new/

https://reviews.llvm.org/D71210





More information about the llvm-commits mailing list