[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