[llvm] r320635 - Remove redundant includes from lib/Target/ARM.

Friedman, Eli via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 13 13:42:49 PST 2017


On 12/13/2017 1:31 PM, Michael Zolotukhin via llvm-commits wrote:
> Author: mzolotukhin
> Date: Wed Dec 13 13:31:17 2017
> New Revision: 320635
>
> URL: http://llvm.org/viewvc/llvm-project?rev=320635&view=rev
> Log:
> Remove redundant includes from lib/Target/ARM.
>
> Modified: llvm/trunk/lib/Target/ARM/ARMMacroFusion.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMMacroFusion.cpp?rev=320635&r1=320634&r2=320635&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/ARM/ARMMacroFusion.cpp (original)
> +++ llvm/trunk/lib/Target/ARM/ARMMacroFusion.cpp Wed Dec 13 13:31:17 2017
> @@ -12,7 +12,6 @@
>   //
>   //===----------------------------------------------------------------------===//
>   
> -#include "ARMMacroFusion.h"
>   #include "ARMSubtarget.h"
>   #include "llvm/CodeGen/MacroFusion.h"
>   #include "llvm/CodeGen/TargetInstrInfo.h"

This isn't right; I mean, it might compile, but it isn't a good idea.

Please revert this bit, and make sure you aren't doing the same thing 
anywhere else.

-Eli

-- 
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project



More information about the llvm-commits mailing list