[PATCH] D63741: [TableGen] Add "InitValue": Handle operands with set bit values in decoder methods

[Nicolas Guillemot via Phabricator via llvm-commits]

nlguillemot marked an inline comment as done.
nlguillemot added inline comments.


================
Comment at: test/TableGen/FixedLenDecoderEmitter/InitValue.td:11
+
+let OutOperandList = (outs), Size = 2 in {
+
----------------
dsanders wrote:
> This is a bit confusing and isn't really doing anything helpful. For this test case, I'd prefer to just have the lets duplicated inside the defs.
> 
> Also, the formatting in this file isn't following the conventions used by most tablegen files (e.g. closing } being in the same column as the 'd' from def)
The formatting is copy/pasted from `test/TableGen/BitOffsetDecoder.td`, so that's my "claim of innocence". Of course we could change it anyways, and clean up `BitOffsetDecoder.td` as well in a separate patch. Thoughts?


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

https://reviews.llvm.org/D63741