nikic added a comment.

I think this (removing GlobalCleanupPM) makes sense at a high level, but this 
is a pretty substantial pipeline change and will need more thorough evaluation 
(e.g. looking at IR diffs on test suite). For example, it is not at all obvious 
to me that moving InstCombine before EarlyCSE is harmless, given the impact 
this may have on one-use heuristics.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145265

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATC... Arthur Eubanks via Phabricator via cfe-commits
    • ... Arthur Eubanks (out until mid-April) via Phabricator via cfe-commits
    • ... Nikita Popov via Phabricator via cfe-commits
    • ... Arthur Eubanks (out until mid-April) via Phabricator via cfe-commits
    • ... Alina Sbirlea via Phabricator via cfe-commits

Reply via email to