[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