[PATCH] D17257: New utility class ReachingPhysDefs for post-ra analysis.

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 9 13:58:19 PST 2016


qcolombet added a comment.

Hi Jonas,

This is NFC for AArch64.

I have a few warnings in the build of the compiler that would need to be fixed:

- ReachingPhysDefs.h:167:3: warning: 'llvm::ReachingPhysDefs' has virtual functions but non-virtual destructor [-Wnon-virtual-dtor]
- AArch64CollectLOH.cpp:210:7: warning: 'ReachingPhysDefs_ADRP' has virtual functions but non-virtual destructor [-Wnon-virtual-dtor]
- AArch64CollectLOH.cpp:225:7: warning: 'ReachingPhysDefs_Others' has virtual functions but non-virtual destructor [-Wnon-virtual-dtor]
- AArch64CollectLOH.cpp:378:30: warning: unused variable 'EndIt' [-Wunused-variable]

Thanks,
-Quentin


http://reviews.llvm.org/D17257





More information about the llvm-commits mailing list