[llvm] r298220 - [CodeGen] Update hasSideEffects comment. NFC.

Ahmed Bougacha via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 19 09:12:45 PDT 2017


Author: ab
Date: Sun Mar 19 11:12:45 2017
New Revision: 298220

URL: http://llvm.org/viewvc/llvm-project?rev=298220&view=rev
Log:
[CodeGen] Update hasSideEffects comment. NFC.

We used to have 3 side effect flags, but as of r222809, we only have
hasSideEffects.  Change the comment to reflect that.

Modified:
    llvm/trunk/include/llvm/Target/Target.td

Modified: llvm/trunk/include/llvm/Target/Target.td
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Target/Target.td?rev=298220&r1=298219&r2=298220&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Target/Target.td (original)
+++ llvm/trunk/include/llvm/Target/Target.td Sun Mar 19 11:12:45 2017
@@ -402,11 +402,8 @@ class Instruction {
                             // If so, make sure to override
                             // TargetInstrInfo::getInsertSubregLikeInputs.
 
-  // Side effect flags - When set, the flags have these meanings:
-  //
-  //  hasSideEffects - The instruction has side effects that are not
-  //    captured by any operands of the instruction or other flags.
-  //
+  // Does the instruction have side effects that are not captured by any
+  // operands of the instruction or other flags?
   bit hasSideEffects = ?;
 
   // Is this instruction a "real" instruction (with a distinct machine




More information about the llvm-commits mailing list