[PATCH] D156346: CodeGen: Disable isCopyInstrImpl if there are implicit operands
    Vitaly Buka via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Jul 26 16:31:17 PDT 2023
    
    
  
vitalybuka added a comment.
In D156346#4537085 <https://reviews.llvm.org/D156346#4537085>, @arsenm wrote:
> In D156346#4536935 <https://reviews.llvm.org/D156346#4536935>, @vitalybuka wrote:
>
>> Just tried, this one does not solve the issue. Or it probably introduce a different miscompile. UBSan report is gone, but the test fails text expectations. With stable ( before D150388 <https://reviews.llvm.org/D150388> ) compiler the test consistently green.
>>
>> D156164 <https://reviews.llvm.org/D156164> also makes the test consistently green.
>
> To clarify did you only apply this patch? You need the whole stack. This one in isolation won’t fix it
yes, only the last one
arc patch does not apply entire stack, probably patches need to be rebased/uploaded
my 2c: if it's entire stack then revert D156381 <https://reviews.llvm.org/D156381> is more appropriate solution
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D156346/new/
https://reviews.llvm.org/D156346
    
    
More information about the llvm-commits
mailing list