[PATCH] D66991: [PowerPC] Fix SH field overflow issue

Jinsong Ji via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 23 08:44:17 PDT 2019


jsji added a comment.

In D66991#1679267 <https://reviews.llvm.org/D66991#1679267>, @jsji wrote:

> You are right, we should make sure we meet SSA constraint when writing the MIR for passes before RA.


BTW: `isSSA()` in MIRParser will always return `true` for MIRs that does NOT have virtual regs, 
I think this is a limitation for now, we might want to extend it to support MIR with all HWregs,
but for now, we should check the code manually and carefully.  Thanks for pointing out.


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

https://reviews.llvm.org/D66991





More information about the llvm-commits mailing list