[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