[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