[PATCH] D40846: Add branch flag on branch instructions

[Leonardo Bianconi via Phabricator via llvm-commits]

lbianc added inline comments.


================
Comment at: lib/Target/PowerPC/PPCInstrInfo.td:1291
 
-  let isCodeGenOnly = 1 in {
+  let isCodeGenOnly = 1, isBranch = 1 in {
     let Pattern = [(brcond i1:$bi, bb:$dst)] in
----------------
nemanjai wrote:
> Why is this needed? This flag is set on line 1261.
Not needed here, didn't realize that parent was setting it. Will be removed.


================
Comment at: lib/Target/PowerPC/PPCInstrInfo.td:4282
 // Defs and Uses are conservative, since we don't know the BO value.
 let PPC970_Unit = 7 in {
+  let isBranch = 1, Defs = [CTR], Uses = [CTR, RM] in {
----------------
nemanjai wrote:
> Why not just set the `isBranch` flag here rather than repeating it in a number of places within this scope?
Ok, moving I will move it to the parent, thanks!


https://reviews.llvm.org/D40846