[PATCH] D73754: [CodeGenPrepare] Make TargetPassConfig required

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 2 11:55:26 PST 2020


MaskRay marked 2 inline comments as done.
MaskRay added inline comments.


================
Comment at: llvm/lib/CodeGen/CodeGenPrepare.cpp:1814
                                   bool &ModifiedDT) {
-  if (!TLI || !DL)
+  if (!DL)
     return false;
----------------
arsenm wrote:
> This should also be a long dead check, but that's a separate change
Thanks. Removed in 44cdae68c3b8068750e3f6edcccad99e091ce922


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73754





More information about the llvm-commits mailing list