[PATCH] D70922: [GlobalISel]: Allow targets to override how to widen constants during legalization

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 3 10:05:43 PST 2019


arsenm added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/GlobalISel/LegalizerInfo.h:1166
+
+  static bool isByteSizedTy(const LLT &Ty);
+
----------------
volkan wrote:
> I think this function should be a member function of LowLevelType.
LLTs should also not be passed by reference 


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

https://reviews.llvm.org/D70922





More information about the llvm-commits mailing list