[PATCH] D116465: [SPIRV 6/6] Add 2 essential passes and the simplest tests

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 3 02:17:37 PST 2022


rengolin added a comment.

In D116465#3292079 <https://reviews.llvm.org/D116465#3292079>, @zuban32 wrote:

> Which stage are you talking about here: InstPrinter, any post-InstructionSelection pass, etc? On gMIR level I don't understand how that'd work if we define VReg %1 in func f1 and then use it in both f1 and f2.

Whatever level you were planning to do the de-duplication. My point is just to replace de-duplication by enforcement (as post-generation pass) to de-duplication by construction (a singleton generator).

Though, I am not super familiar with MIR's representation, as that was introduced after I stopped working on back-ends directly, so don't take my word on that.

Perhaps @arsenm can chime in?


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

https://reviews.llvm.org/D116465



More information about the llvm-commits mailing list