kparzysz added a comment.

I'm not sure if it's worth defining these macros especially if this is the only 
place where they are used.  We'd need a more coordinated effort to replace all 
32's with named constants where it refers to a register size, and I'm not even 
sure if we could classify whether a given 32 is tied to a register size or not.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75638



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to