[PATCH] D149288: [X86] Introduce a large data threshold for the medium code model

[Thomas Köppe via Phabricator via llvm-commits]

tkoeppe added inline comments.


================
Comment at: llvm/tools/llc/llc.cpp:599
       Target->setCodeModel(*CM_IR);
+    if (auto LDT = codegen::getExplicitLargeDataThreshold())
+      Target->setLargeDataThreshold(*LDT);
----------------
Can we avoid the implicit check for 0 here and spell out `; LDT > 0`? Does LLVM require C++17?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149288