[PATCH] Add more ChangeTo* functions to MachineOperand

hfinkel at anl.gov hfinkel at anl.gov
Tue May 5 23:56:25 PDT 2015


LGTM.


================
Comment at: include/llvm/CodeGen/MachineOperand.h:558
@@ +557,3 @@
+  /// ChangeToES - Replace this operand with a new external symbol operand.
+  void ChangeToES(const char *SymName, unsigned char TargetFlags = 0);
+
----------------
arsenm wrote:
> hfinkel wrote:
> > Please write out ChangeToExternalSymbol
> This was to match the name of CreateES
Alright (that's unfortunate ;) ).

http://reviews.llvm.org/D9505

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list