[llvm-commits] [llvm] r76686 - in /llvm/trunk: include/llvm/Target/TargetAsmInfo.h lib/Target/ELFTargetAsmInfo.cpp lib/Target/TargetAsmInfo.cpp

Anton Korobeynikov anton at korobeynikov.info
Sat Aug 1 05:26:40 PDT 2009


Hello, Chris

> Now that RelocBehaviour() is never overloaded, it doesn't need to be
> virtual.  Just inline it into its two current call sites in preparation
> for simplifying the code.
Could you please add a comment there, that stuff should be reverted
back some day? AFAIR there are other targets (except alpha and ia64)
which will need to override this logic.

-- 
With best regards, Anton Korobeynikov
Faculty of Mathematics and Mechanics, Saint Petersburg State University




More information about the llvm-commits mailing list