[PATCH] D124217: [AMDGPU] Allow finer grain control of an unaligned access speed

Ivan Kosarev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 18 00:41:57 PDT 2022


kosarev added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:17017
     // 1. Check the alignment / fast memory access constraint.
-    bool IsFast = false;
+    unsigned IsFast = 0;
     if (!TLI.allowsMemoryAccess(*DAG->getContext(), DAG->getDataLayout(), ResVT,
----------------
Shouldn't we also rename them all to something like `Fastness`?


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

https://reviews.llvm.org/D124217



More information about the llvm-commits mailing list