[PATCH] D157685: [SDAG] Don't transfer !range metadata without !noundef to SDAG (PR64589)

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 11 07:21:07 PDT 2023


nikic added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp:4168
+    return nullptr;
+  return I.getMetadata(LLVMContext::MD_range);
+}
----------------
arsenm wrote:
> Alternatively could you just insert a freeze if it's not noundef?
It's not possible to use range metadata past a freeze, so that would give you the same end result (plus additional regressions from having the freeze).


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

https://reviews.llvm.org/D157685



More information about the llvm-commits mailing list