[PATCH] D60741: Remove some more unused headers from MachineFunction.h and friends.

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 15 17:23:53 PDT 2019


dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.

Sounds good to me.
How'd you test that the MachineMemOperand.h changes were OK? Looks like it doesn't have an implementation file, so I'm not sure it's included first in any file - which means it's harder to tell if removing headers from it is valid/keeps it standalone (since its inclusion context might be allowing it to get away without including necessary headers)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60741





More information about the llvm-commits mailing list