<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 26, 2022 at 8:17 PM Matthias Braun via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">MatzeB added inline comments.<br>
<br>
<br>
================<br>
Comment at: llvm/include/llvm/Target/TargetMachine.h:137<br>
+  /// registers, except that variadic defs will be allocated vregs.<br>
+  virtual bool usesPhysRegsForValues() const { return true; }<br>
+<br>
----------------<br>
arsenm wrote:<br>
> echristo wrote:<br>
> > I'm curious what this changes here versus using LLVMTargetMachine. It's been a while since I've looked.<br>
> It doesn't really change anything, but this one doesn't fit the theme of the what LLVMTargetMachine adds. This is core target information and it doesn't make much since to have it in LLVMTargetMachine, which only adds pass machinery controls.<br>
This seems like an implementation detail of `lib/CodeGen` so I think it should stay withint `LLVMTargetMachine`.<br></blockquote><div><br></div><div>Thanks for the memory as well. I agree with everything here. </div></div></div>