[PATCH] D57493: [RISCV] Put data smaller than integer type to small data section

Shiva Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 1 17:37:48 PST 2019


shiva0217 marked an inline comment as done.
shiva0217 added inline comments.


================
Comment at: lib/Target/RISCV/RISCVTargetObjectFile.cpp:39
+      SSThreshold = 8;
+  }
+}
----------------
apazos wrote:
> It seems we force a default value (4 or 8) even when the threshold flag is off (value is 0)?
> So how to disable the optimization?
Hi Ana,
You're right. I've updated the revision to address the issue.
Thanks for the catch.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57493





More information about the llvm-commits mailing list