[PATCH] D82916: LIS: fix handleMove to properly extend main range

Valery Pykhtin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 7 09:15:52 PDT 2020


vpykhtin added a comment.

I understood your patch. Generally I think patching LIS with cleared undef flags ins't right at first place because the semantic of register lifetime is ruined. After the first move there should be an undef flag set at the %1.sub2 = IMPLICIT_DEF instruction which would break lives of all subregs live at this point. Can we drop updating LIS during scheduling and recreate it from scratch? Or may be fully recreate only the intervals for the registers involved in moves when the undef flag is patched after scheduling?


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

https://reviews.llvm.org/D82916





More information about the llvm-commits mailing list