<div dir="ltr">Hi Stefan,<div><br></div><div>If you want to get this reviewed, I would create a phabricator review on <a href="http://reviews.llvm.org">reviews.llvm.org</a>, clearly describing what you are trying to do in the patch, etc.<br></div><div><br></div><div>You reference a bug number, but the bug is just titled "enhancements to the LLVM SPARC backend"</div><div><br></div><div>:)</div><div>That is not very helpful in figuring out the goal of the patch, etc.</div><div><br></div><div>Staring at the patch, it looks like it does a bunch of things. Some bug fixes, some improvements.</div><div> In general, you want to break up the different functional things it does into individual patches so it is easy to review.</div><div><br></div><div>For example, i see:<br><div> --- lib/Target/Sparc/SparcInstrInfo.td  2015-12-11 10:20:16.000000000 -0900</div><div> +++ lib/Target/Sparc/SparcInstrInfo.td  2016-07-02 19:46:36.045655245 -0800</div><div> @@ -348,7 +348,7 @@</div><div>  }</div><div><br></div><div>  // GETPCX for PIC</div><div> -let Defs = [O7] in {</div><div> +let Defs = [O7], hasSideEffects = 1 in {</div><div>    def GETPCX : Pseudo<(outs getPCX:$getpcseq), (ins), "$getpcseq", [] >;</div><div>  }</div><div><br></div><div> ###</div></div><div><br></div><div><br></div><div>This looks like a bug fix, while other parts of the patch look like cpu renamings, etc.</div><div><br></div><div>(I'm just guessing here).</div><div><br></div><br></div>