[PATCH] D91762: [dfsan] Remove deadcode from DFSanFunction::get*TLS*()
Matt Morehouse via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 19 06:25:44 PST 2020
morehouse accepted this revision.
morehouse added inline comments.
This revision is now accepted and ready to land.
================
Comment at: llvm/lib/Transforms/Instrumentation/DataFlowSanitizer.cpp:1069
+ assert(DFS.RetvalTLS);
+ return RetvalTLSPtr = DFS.RetvalTLS;
}
----------------
Since these functions are now just assigning and returning variables, can we completely remove the functions and the `*TLSPtr` variables, and just use `DFS.*TLS` directly?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91762/new/
https://reviews.llvm.org/D91762
More information about the llvm-commits
mailing list