[PATCH] D67065: [RISCV] Define __riscv_cmodel_medlow and __riscv_cmodel_medany correctly

Sam Elliott via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 2 12:34:48 PDT 2019


lenary added inline comments.


================
Comment at: clang/lib/Basic/Targets/RISCV.cpp:91
   Builder.defineMacro("__riscv_xlen", Is64Bit ? "64" : "32");
   // TODO: modify when more code models are supported.
+  StringRef CodeModel = getTargetOpts().CodeModel;
----------------
This TODO can probably be deleted - now it's more obvious where to add more code when we support more code models.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67065





More information about the cfe-commits mailing list