[PATCH] D66079: [SimplifyLibCalls] Add dereferenceable bytes from known callsites [WIP]

[Dávid Bolvanský via Phabricator via llvm-commits]

xbolva00 marked an inline comment as done.
xbolva00 added a comment.

(I havent run test suite yet, I expect many test files need to be updated.. I will do it tommorow)



================
Comment at: lib/Transforms/Utils/SimplifyLibCalls.cpp:193
+  for (unsigned ArgNo : ArgNos) {
+    if (CI->getDereferenceableBytes(ArgNo + 1) < DerefBytes) {
+      CI->removeParamAttr(ArgNo, Attribute::Dereferenceable);
----------------
if (CI->getDereferenceableBytes(**ArgNo + 1**) < DerefBytes) {

quite surprising :) spent some time on find this :D


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

https://reviews.llvm.org/D66079