[PATCH] D81461: GlobalISel: Fix double printing new instructions in legalizer

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 9 06:00:02 PDT 2020

Previous message: [PATCH] D81461: GlobalISel: Fix double printing new instructions in legalizer
Next message: [PATCH] D81461: GlobalISel: Fix double printing new instructions in legalizer
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

arsenm added a comment.

It seems both the MachineFunction and MachineIRBuilder have insertion observers; this seems redundant. I think the machine function observers should be removed?


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

https://reviews.llvm.org/D81461

Previous message: [PATCH] D81461: GlobalISel: Fix double printing new instructions in legalizer
Next message: [PATCH] D81461: GlobalISel: Fix double printing new instructions in legalizer
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list