[PATCH] D83175: [X86] Fix a bug that when lowering byval argument

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 5 09:02:19 PDT 2020


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:4045
         isByVal = false;
+        hasCopy = true;
       } else {
----------------
Why do we need a new variable? Can we just pass isByVal to the function?


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

https://reviews.llvm.org/D83175





More information about the llvm-commits mailing list