[PATCH] D59375: Allow unordered loads to be considered invariant in CodeGen
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 19 11:20:50 PDT 2019
reames marked an inline comment as done.
reames added inline comments.
================
Comment at: lib/CodeGen/MachineInstr.cpp:1318
+ // instruction. Such an instruction is technically an invariant load,
+ // but the caller code would need updated to expect that.
+ return false;
----------------
jlebar wrote:
> "need to be updated"?
I left the wording here because this is a possible change with possible downsides design wise. I don't what to give the impression it's a straight forward change.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59375/new/
https://reviews.llvm.org/D59375
More information about the llvm-commits
mailing list