Don't require a specific kind of IRBuilder for TargetLowering hooks. This allows us to drop the IRBuilder.h include from TargetLowering.h.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/include/llvm/CodeGen/TargetLowering.h | ||
---|---|---|
42–43 | I don't suppose this will let us drop the IRBuilder.h include and just use forward declarations? TargetLowering.h is a pretty heavy include and anything to reduce it would be very useful. |
llvm/include/llvm/CodeGen/TargetLowering.h | ||
---|---|---|
42–43 | Yes, that's exactly the goal here :) I just wanted to separate the API changing part of the change first. Otherwise this should be matter of moving a couple default implementations out of line and lots of include fixup in other files. |
llvm/include/llvm/CodeGen/TargetLowering.h | ||
---|---|---|
42–43 | Sorry I obviously didn't fully read the summary :) |
Just to be certain - is this actually going to work? After fixing the implicit dependencies will you be able to remove the include?
I don't suppose this will let us drop the IRBuilder.h include and just use forward declarations? TargetLowering.h is a pretty heavy include and anything to reduce it would be very useful.