[PATCH] D149438: Remove some includes that shouldn't be needed any longer

Bjorn Pettersson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 16 07:10:28 PDT 2023


bjope added a comment.

In D149438#4343022 <https://reviews.llvm.org/D149438#4343022>, @aeubanks wrote:

> if you're fairly certain these are correct, I'd say no need for a review for include cleanups in the future :)

Ok. Thanks. I've pushed without review on some rare occasions, but usually prefer to get a second opinion to make sure I do not mess up things.
But I'll try to avoid taking up valuable time for those making lots of reviews in this project for trivial patches like this one.

I was slightly worried that I could have missed something with how the LLVM libs works (that removing things from Scalar.cpp could break something).
But since code compiles and check-all passes, then I guess that it should be fine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149438



More information about the llvm-commits mailing list