[PATCH] D94437: [AArch64][GlobalISel] Add support for FCONSTANT of FP128 type

[Muhammad Asif Manzoor via Phabricator via llvm-commits]

Asif added inline comments.


================
Comment at: llvm/lib/Target/AArch64/GISel/AArch64InstructionSelector.cpp:2318-2319
       // can be sure tablegen works correctly and isn't rescued by this code.
-      if (I.getOperand(1).getFPImm()->getValueAPF().isExactlyValue(0.0))
+      // 0.0 is not covered by tablegen for FP128. So we will handle this 
+      // scenario in the code here.
+      if (DefSize != 128 && 
----------------
arsenm wrote:
> Why not just fix this?
I haven't worked with the tablegen before. Please point to the source files so I can look into it and fix it. 


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

https://reviews.llvm.org/D94437