[PATCH] D88335: [BuildLibCalls] Add noalias for strcat and stpcpy

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 26 01:12:09 PDT 2020


xbolva00 added a comment.

In D88335#2296000 <https://reviews.llvm.org/D88335#2296000>, @jdoerfert wrote:

> FWIW, `noalias` on these arguments is, by itself, not doing anything. It does something if you know how much memory is accessed by the function through such an argument *for sure* and what other memory is accessed for sure. Such memory regions are not overlapping. (This is way too vague actually but I guess it doesn't matter for the gist.)

so, the patch as is, is OK for you all?


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

https://reviews.llvm.org/D88335



More information about the llvm-commits mailing list