[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