[PATCH] D90107: [TableGen] [AMDGPU] Add !sub bang operator for subtraction

Paul C. Anagnostopoulos via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 28 07:37:01 PDT 2020


Paul-C-Anagnostopoulos marked 2 inline comments as done.
Paul-C-Anagnostopoulos added inline comments.


================
Comment at: llvm/lib/TableGen/TGParser.cpp:1259
+            Code != BinOpInit::SRL && Code != BinOpInit::SHL &&
+            Code != BinOpInit::MUL)
           ArgType = Resolved;
----------------
madhur13490 wrote:
> I think the condition can be moved to a separate function before it becomes unmanageable.
There are complex conditions like this all over TableGen.


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

https://reviews.llvm.org/D90107



More information about the llvm-commits mailing list