[PATCH] D60580: [GlobalISel] Enable CSE in the IRTranslator & legalizer for -O0 with constants only

Aditya Nandakumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 12 16:33:42 PDT 2019


aditya_nandakumar added a comment.

Looks reasonable. Could you please split out the CSEConfigBase into it's own commit?
Also I only see the includes for the CSEConfigBase.h but not the header file in this patch. Maybe you forgot to include it?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60580





More information about the llvm-commits mailing list