[PATCH] D60696: [TableGen] New default operand "undef_tied_input".

Puyan Lotfi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 30 12:56:57 PDT 2019


plotfi added subscribers: bogner, plotfi.
plotfi added a comment.

I am seeing a build warning now. Is this a bug?

  	/mnt/nvme0/llvm-project/llvm/utils/TableGen/GlobalISelEmitter.cpp: In member function ‘llvm::Error {anonymous}::GlobalISelEmitter::importDefaultOperandRenderers({anonymous}::action_iterator, {anonymous}::RuleMatcher&, {anonymous}::BuildMIAction&, llvm::DagInit*) const’:
  /mnt/nvme0/llvm-project/llvm/utils/TableGen/GlobalISelEmitter.cpp:3816:9: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
           if (DefaultDagOperator->getDef()->isSubClassOf("ValueType"))
           ^~
  /mnt/nvme0/llvm-project/llvm/utils/TableGen/GlobalISelEmitter.cpp:3819:11: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’
             DefaultOp = DefaultDagOp->getArg(0);
             ^~~~~~~~~

@bogner


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60696/new/

https://reviews.llvm.org/D60696





More information about the llvm-commits mailing list