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

Bevin Hansson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 10 07:19:27 PST 2019


ebevhan added a comment.

In D71210#1776911 <https://reviews.llvm.org/D71210#1776911>, @jmolloy wrote:

> In fact, one concrete improvement we could do with this CL (which clearly found everything we needed to patch) is introduce a typedef for "A bitmask of functional units". This would allow us to find all the instances again in future, and highlight the point of the bitwidth to users.


I considered that. I could make that change.


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